-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make cumsum(A)
compute a summed-area table
#26412
Comments
More than just |
We should at least support it when |
@yurivish Something like this?
It works even for subsets of
I can work this into a PR. Just need some time to get acquainted to JuliaLang PR policies, so devs can assign me this if you want. |
GitHub only allows assignment to members of an organization, but you can certainly work on this if you like. Keep in mind that one will want to compute the summed area table in an order which is cache-friendly (i.e. compute along columns first for dense arrays). |
I think that definition of
|
Yes, it does seem like a nice definition. If I'm understanding correctly, it's compatible with computing a summed area table as well, assuming that the default is |
Hi, If the order of summation doesn't matter then I think simply calculating the cumulative sum will be good, right? If nobody is working on this I would like to contribute. |
No, I would say just try it and ask for help on the #my-first-pr channel on julialang.slack.com if you need help with anything. |
The idea has been discussed in a few places (e.g. #20041 (comment)) to make
cumsum(A)
return a summed-area table for multidimensional arrays.The 1D case is already a summed-area table, since that's just the cumulative sum along the one dimension.
Happily, the appropriate deprecation for
cumsum
— deprecating it forAbstractArray
s with nodims
argument — is already in place, so this is just a tracking issue for the idea.cc: @simonbyrne
The text was updated successfully, but these errors were encountered: