Open tpetricek opened 10 years ago
After I wrote that, I swapped the order of the summaryOp
and sweepOp
parameters to better reflect the order in which they're applied in the zipInto
call, e.g.
let sweepLevel level sweepOp summaryOp series = ...
and
f?N |> sweepLevel fst (/) Stats.sum
I definitely like it, but I agree that it adds to the naming confusion.
Include something like this (inspired by R "sweep" operation):
So, for example, to normalize each values (divide by the sum), we can write:
(I think the level aggregation functions have a bit confusing naming, so adding this contributes to that, but it is a super useful function... Perhaps there is some better naming scheme we could follow!)