Skip to content

ENH: add cummax/cummin/cumprod support for arrow dtypes #54574

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

Merged
merged 4 commits into from
Aug 17, 2023

Conversation

lukemanley
Copy link
Member

un-xfails 186 tests which saves a chunk of time in test_arrow.py

The new pyarrow cumulative functions are only available starting in pyarrow version 13 which is not yet released but is currently tested in CI via the pyarrow nightly job. We could hold off on merging until pyarrow 13 is released if there are any concerns.

@lukemanley lukemanley added Enhancement Arrow pyarrow functionality labels Aug 16, 2023
@lukemanley lukemanley added this to the 2.2 milestone Aug 16, 2023
@jbrockmendel
Copy link
Member

un-xfails 186 tests which saves a chunk of time in test_arrow.py

nice!

i'd also be OK with implementing it for pyarrow>=13 and leaving it xfailed for earlier versions

@mroeschke
Copy link
Member

I would also be okay backporting this to 2.1

@mroeschke mroeschke modified the milestones: 2.2, 2.1 Aug 16, 2023
@mroeschke mroeschke merged commit cf7f0af into pandas-dev:main Aug 17, 2023
@mroeschke
Copy link
Member

Thanks @lukemanley

meeseeksmachine pushed a commit to meeseeksmachine/pandas that referenced this pull request Aug 17, 2023
mroeschke pushed a commit that referenced this pull request Aug 17, 2023
…pport for arrow dtypes) (#54603)

Backport PR #54574: ENH: add cummax/cummin/cumprod support for arrow dtypes

Co-authored-by: Luke Manley <[email protected]>
mroeschke pushed a commit to mroeschke/pandas that referenced this pull request Aug 18, 2023
…4574)

* ENH: add cumax, cumin, cumprod support to ArrowExtensionArray

* whatsnew

* move whatsnew to 2.1.0
@lukemanley lukemanley deleted the arrow-cumulative-functions branch September 6, 2023 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Arrow pyarrow functionality Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: Cumprod failing with type float[pyarrow]
3 participants