-
-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
Maybe change this line:
From
(-reduce [this rfn] (-reduce (range cnt) #(rfn %1 (idx->val %2))))
To something like:
(-reduce [this rfn]
(if (zero? cnt)
(rfn)
(-reduce (hamf/range 1 cnt) #(rfn %1 (idx->val %2)) (idx->val 0))))
Test:
(reduce + (dt/reify-reader :float64 3 (partial nth [3 2 5])))
-
Before change: 7, after change: 10
-
Note: there is a similar issue in tech.v3.datatype.emap1-vec
Metadata
Metadata
Assignees
Labels
No labels