update arraymancer dependency, remove toHashSet

`toHashSet` nowadays lives in arraymancer, courtesy of @AngelEzquerra.
SciNim · May 26, 2024 · ffe835e · ffe835e
1 parent 1f5294d
commit ffe835e
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/datamancer.nimble b/datamancer.nimble
@@ -11,7 +11,7 @@ srcDir        = "src"
 
 requires "nim >= 1.2.0"
 requires "https://github.com/Vindaar/seqmath >= 0.2.1"
-requires "arraymancer >= 0.7.28"
+requires "arraymancer >= 0.7.30"
 
 task test, "Run standard tests":
   exec "nim c -r tests/testDf.nim"

diff --git a/src/datamancer/dataframe.nim b/src/datamancer/dataframe.nim
@@ -2003,11 +2003,6 @@ proc innerJoin*[C: ColumnLike](dfs: varargs[DataTable[C]], by: string): DataTabl
     for i in 2 ..< dfs.len:
       result = result.innerJoin(dfs[i], by)
 
-proc toHashSet*[T](t: Tensor[T]): HashSet[T] =
-  ## Internal helper to convert a tensor to a `HashSet`
-  for el in t:
-    result.incl el
-
 proc group_by*[C: ColumnLike](df: DataTable[C], by: varargs[string], add = false): auto =
   ## Returns a grouped data frame grouped by all unique keys in `by`.
   ##