You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mahout.apache.org by ro...@apache.org on 2010/03/03 15:43:30 UTC
svn commit: r918493 -
/lucene/mahout/trunk/math/src/main/java/org/apache/mahout/math/DenseVector.java
Author: robinanil
Date: Wed Mar 3 14:43:30 2010
New Revision: 918493
URL: http://svn.apache.org/viewvc?rev=918493&view=rev
Log:
Introduced a bug, Missed out resetting length squared on fast densevector assign
Modified:
lucene/mahout/trunk/math/src/main/java/org/apache/mahout/math/DenseVector.java
Modified: lucene/mahout/trunk/math/src/main/java/org/apache/mahout/math/DenseVector.java
URL: http://svn.apache.org/viewvc/lucene/mahout/trunk/math/src/main/java/org/apache/mahout/math/DenseVector.java?rev=918493&r1=918492&r2=918493&view=diff
==============================================================================
--- lucene/mahout/trunk/math/src/main/java/org/apache/mahout/math/DenseVector.java (original)
+++ lucene/mahout/trunk/math/src/main/java/org/apache/mahout/math/DenseVector.java Wed Mar 3 14:43:30 2010
@@ -116,7 +116,14 @@
lengthSquared = -1.0;
values[index] = value;
}
-
+
+ @Override
+ public Vector assign(double value) {
+ this.lengthSquared = -1;
+ Arrays.fill(values, value);
+ return this;
+ }
+
@Override
public Vector assign(Vector other, BinaryFunction function) {
if (other.size() != size()) {
@@ -326,12 +333,7 @@
values[element.index()] += element.get();
}
}
-
- @Override
- public Vector assign(double value) {
- Arrays.fill(values, value);
- return this;
- }
+
@Override
public double dot(Vector x) {