Merge pull request #99 from realratchet/master

Boolean infer more flexible

tablite/_nimlite/infertypes.nim

@@ -16,11 +16,11 @@ proc inferNone*(str: ptr string): PY_NoneType {.inline.} =
     raise newException(ValueError, "not a none")
 
 proc inferBool*(str: ptr string): bool {.inline.} =
-    let sstr = str[]
+    let sstr = str[].toLower()
 
-    if sstr in ["True", "true"]:
+    if sstr == "true":
         return true
-    elif sstr in ["False", "false"]:
+    elif sstr == "false":
         return false
 
     raise newException(ValueError, "not a boolean value")

tablite/core.py

@@ -64,7 +64,7 @@ def from_file(
         text_escape_openings="",
         text_escape_closures="",
         tqdm=_tqdm,
-    ):
+    ) -> "Table":
         """
         reads path and imports 1 or more tables
 

tablite/version.py

@@ -1,3 +1,3 @@
-major, minor, patch = 2023, 8, 1
+major, minor, patch = 2023, 8, 2
 __version_info__ = (major, minor, patch)
 __version__ = ".".join(str(i) for i in __version_info__)

tests/data/booleans.csv

@@ -0,0 +1,2 @@
+lc_false,lc_true,py_false,py_true,xl_false,xl_true
+false,true,False,True,FALSE,TRUE

tests/test_filereader_formats.py

@@ -658,3 +658,9 @@ def test_header_offset_ods():
 
     assert list(tbl.columns.keys()) == ["header"]
     assert list(tbl["header"]) == [1, 2, 3, 4, 5]
+
+def test_booleans():
+    tbl = Table.from_file(Path(__file__).parent / "data" / "booleans.csv")
+
+    assert len(tbl) == 1
+    assert next(tbl.rows) == [False, True, False, True, False, True]