From 4a264a102af41052a4c2cdd49bf460ce28d1c81e Mon Sep 17 00:00:00 2001 From: Vindaar Date: Sun, 26 May 2024 13:30:10 +0200 Subject: [PATCH] update arraymancer dependency, remove `toHashSet` `toHashSet` nowadays lives in arraymancer, courtesy of @AngelEzquerra. --- datamancer.nimble | 2 +- src/datamancer/dataframe.nim | 5 ----- 2 files changed, 1 insertion(+), 6 deletions(-) diff --git a/datamancer.nimble b/datamancer.nimble index 3bb07ca..5fd8152 100644 --- 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 index 312ac02..059c68c 100644 --- 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`. ##