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


Reply via email to