Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[WIP, ENH] Adds cumulative methods to ea #28509
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
[WIP, ENH] Adds cumulative methods to ea #28509
Changes from 4 commits
2897723
a54e1b4
10abc0f
c2d7592
2c149c0
79cea11
12a5ca3
dc959f4
bcfb8a8
9a8f4ec
5d837d9
9e9f0c3
a1a1cb2
6d967ad
73363bf
0d9a3d5
84a7d81
ce6869d
3b5d1d8
0337cb0
f0722f5
99baa1b
fa35b14
43fca7c
7bd6378
185510b
2ef9ebb
7d898bd
09b42be
af0dd24
bc9a36a
38454a3
5ecfa51
8d62594
5f3b624
06d1286
7efcb5f
ae5f969
f7e3f4f
aa99927
9cab6d9
b3ae864
52e6486
99fb664
d339250
be6f974
a902f4e
64afb5b
1e5d77b
c95b490
dc669de
08475a4
67fa99a
a36632b
b3d3c81
f8f6367
ad6773d
663c301
e17f3a0
8cb66f9
4f953cf
b33a5df
18ec178
305bdc7
56bfb23
63db854
9c91c55
6ba3ca9
f2a49b3
386fa39
55de384
6a5b7f8
9c63c64
84dd141
2f23499
a5b30e6
d22c8a0
a5866c7
8255457
483b608
150fd3b
80e2dc6
dceab99
1c14f18
e20501a
53147c4
597e978
5ebe8ea
32367c0
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May be for numpy compatibility (axis, dtype, out).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are your expectations adding this? Do you purely want to have it (for this ticket) as accessor, which we'll ignore? If not, I'd guess the impact of axis is None, but dtype might be interesting. I.e for cumsum and integer dtypes, we could also provide the target output type, so not defaulting to (U)Int64. But I don't know if this should be part of this issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The idea would be for something like
np.cumsum(pd.array([1, 2]))
to return an IntegerArray. I'm not sure what all is required for that to work.