You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mahout.apache.org by sm...@apache.org on 2013/11/29 20:54:01 UTC

svn commit: r1546636 - in /mahout/trunk/integration/src: main/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModel.java test/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModelTest.java

Author: smarthi
Date: Fri Nov 29 19:54:00 2013
New Revision: 1546636

URL: http://svn.apache.org/r1546636
Log:
MAHOUT-1285: Return NaN if a string

Modified:
    mahout/trunk/integration/src/main/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModel.java
    mahout/trunk/integration/src/test/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModelTest.java

Modified: mahout/trunk/integration/src/main/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModel.java
URL: http://svn.apache.org/viewvc/mahout/trunk/integration/src/main/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModel.java?rev=1546636&r1=1546635&r2=1546636&view=diff
==============================================================================
--- mahout/trunk/integration/src/main/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModel.java (original)
+++ mahout/trunk/integration/src/main/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModel.java Fri Nov 29 19:54:00 2013
@@ -148,7 +148,7 @@ public class MapBackedARFFModel implemen
     if (isNumeric(data)) {
       return Double.parseDouble(data);
     }
-    return 0.0;
+    return Double.NaN;
   }
 
   public static boolean isNumeric(String str) {

Modified: mahout/trunk/integration/src/test/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModelTest.java
URL: http://svn.apache.org/viewvc/mahout/trunk/integration/src/test/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModelTest.java?rev=1546636&r1=1546635&r2=1546636&view=diff
==============================================================================
--- mahout/trunk/integration/src/test/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModelTest.java (original)
+++ mahout/trunk/integration/src/test/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModelTest.java Fri Nov 29 19:54:00 2013
@@ -45,7 +45,7 @@ public class MapBackedARFFModelTest exte
     ARFFModel model = new MapBackedARFFModel();
     model.addLabel("b1shkt70694difsmmmdv0ikmoh", 77);
     model.addType(77, ARFFType.REAL);
-    assertTrue(0.0 == model.getValue("b1shkt70694difsmmmdv0ikmoh", 77));
+    assertTrue(Double.isNaN(model.getValue("b1shkt70694difsmmmdv0ikmoh", 77)));
   }
 
   @Test