You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mahout.apache.org by sr...@apache.org on 2009/12/12 23:48:57 UTC

svn commit: r889994 - /lucene/mahout/trunk/matrix/src/main/java/org/apache/mahout/matrix/SparseVector.java

Author: srowen
Date: Sat Dec 12 22:48:57 2009
New Revision: 889994

URL: http://svn.apache.org/viewvc?rev=889994&view=rev
Log:
Clear 0.0 entries on set in SparseVector

Modified:
    lucene/mahout/trunk/matrix/src/main/java/org/apache/mahout/matrix/SparseVector.java

Modified: lucene/mahout/trunk/matrix/src/main/java/org/apache/mahout/matrix/SparseVector.java
URL: http://svn.apache.org/viewvc/lucene/mahout/trunk/matrix/src/main/java/org/apache/mahout/matrix/SparseVector.java?rev=889994&r1=889993&r2=889994&view=diff
==============================================================================
--- lucene/mahout/trunk/matrix/src/main/java/org/apache/mahout/matrix/SparseVector.java (original)
+++ lucene/mahout/trunk/matrix/src/main/java/org/apache/mahout/matrix/SparseVector.java Sat Dec 12 22:48:57 2009
@@ -86,7 +86,11 @@
   @Override
   public void setQuick(int index, double value) {
     lengthSquared = -1.0;
-    values.put(index, value);
+    if (value == 0.0) {
+      values.removeKey(index);
+    } else {
+      values.put(index, value);
+    }
   }
 
   @Override
@@ -372,7 +376,6 @@
 
     @Override
     public boolean apply(int key, double value) {
-      lengthSquared = -1.0;
       v.set(key, value + v.get(key));
       return true;
     }