Merge pull request #1 from Breeding-Insight/bug/BI-2055

[BI-2055] - Remove missingValueString tablesaw values
Breeding-Insight · Jan 15, 2025 · feeef35 · feeef35
2 parents 10ebd2f + 2986e5d
commit feeef35
Show file tree

Hide file tree

Showing 4 changed files with 110 additions and 118 deletions.
diff --git a/core/src/main/java/tech/tablesaw/io/TypeUtils.java b/core/src/main/java/tech/tablesaw/io/TypeUtils.java
@@ -22,18 +22,10 @@
 @Immutable
 public final class TypeUtils {
 
-  /** Strings representing missing values in, for example, a CSV file that is being imported */
-  private static final String missingInd1 = "NaN";
-
-  private static final String missingInd2 = "*";
-  private static final String missingInd3 = "NA";
-  private static final String missingInd4 = "null";
-  private static final String missingInd5 = "N/A";
-
   // No default missing indicators
   // TODO: Allow this to be configurable?
   public static final ImmutableList<String> MISSING_INDICATORS =
-      ImmutableList.of(missingInd1, missingInd2, missingInd4, missingInd5);
+      ImmutableList.of();
 
   /** Private constructor to prevent instantiation */
   private TypeUtils() {}

diff --git a/core/src/test/java/tech/tablesaw/io/csv/CsvReaderTest.java b/core/src/test/java/tech/tablesaw/io/csv/CsvReaderTest.java
@@ -528,7 +528,7 @@ void testWithMissingValue2() throws IOException {
 
     Table t = Table.read().csv("../data/missing_values2.csv");
     assertEquals(1, t.stringColumn(0).countMissing());
-    assertEquals(1, t.numberColumn(1).countMissing());
+    assertEquals(0, t.numberColumn(1).countMissing());
     assertEquals(0, t.numberColumn(2).countMissing());
   }