[r] Deal with int64 corner case (#607)

* [r] Deal with int64 corner case * Update NEWS and roll micro release [ci skip]
TileDB-Inc · Oct 31, 2023 · 449fa31 · 449fa31
1 parent 62c152f
commit 449fa31
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,6 +1,6 @@
 Package: tiledb
 Type: Package
-Version: 0.21.1.10
+Version: 0.21.1.11
 Title: Universal Storage Engine for Sparse and Dense Multidimensional Arrays
 Authors@R: c(person("TileDB, Inc.", role = c("aut", "cph")),
  person("Dirk", "Eddelbuettel", email = "dirk@tiledb.com", role = "cre"))

diff --git a/NEWS.md b/NEWS.md
@@ -20,6 +20,8 @@
 
 * Use of TileDB Embedded was upgraded to release 2.17.3 (#606)
 
+* Factor variables with (unlikely) int64 indices are supported (#607)
+
 ## Bug Fixes
 
 * The DESCRIPTION file now correctly refers to macOS 10.14 (#596)

diff --git a/R/TileDBArray.R b/R/TileDBArray.R
@@ -1008,6 +1008,9 @@ setMethod("[", "tiledb_array",
 
                           col <- col + 1L # adjust for zero-index C/C++ layer
 
+                          if (inherits(col, "integer64")) # can happen when Python writes
+                              col <- as.integer(col)
+
                           ## special case from schema evolution could have added twice so correct
                           if (min(col, na.rm=TRUE) == 2 && max(col, na.rm=TRUE) == length(dct) + 1)
                               col <- col - 1L