Skip to content
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

Correctly handle output for GroupBy.apply when chunk results are reindexed series #15109

Merged
merged 6 commits into from
Mar 13, 2024

Conversation

brandon-b-miller
Copy link
Contributor

Closes #15084

@brandon-b-miller brandon-b-miller added bug Something isn't working Python Affects Python cuDF API. non-breaking Non-breaking change labels Feb 21, 2024
@brandon-b-miller brandon-b-miller requested a review from a team as a code owner February 21, 2024 19:02
python/cudf/cudf/core/groupby/groupby.py Outdated Show resolved Hide resolved
python/cudf/cudf/tests/test_groupby.py Outdated Show resolved Hide resolved
@brandon-b-miller
Copy link
Contributor Author

some possibly transient CI failures here - going to try just rerunning.

@bdice
Copy link
Contributor

bdice commented Mar 10, 2024

The current CI failures in https://github.com/rapidsai/cudf/actions/runs/8202783961/job/22449311998?pr=15109 are blocked by #15265.

@brandon-b-miller
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 39a365b into rapidsai:branch-24.04 Mar 13, 2024
74 checks passed
wence- added a commit to wence-/cudf that referenced this pull request Mar 13, 2024
This was required by rapidsai#15109, but removed by the changes in rapidsai#15145 and
the merge order was such that they weren't tested against each other.
@wence- wence- mentioned this pull request Mar 13, 2024
3 tasks
rapids-bot bot pushed a commit that referenced this pull request Mar 13, 2024
This was required by #15109, but removed by the changes in #15145 and the merge order was such that they weren't tested against each other.

Authors:
  - Lawrence Mitchell (https://github.com/wence-)

Approvers:
  - Bradley Dice (https://github.com/bdice)
  - GALI PREM SAGAR (https://github.com/galipremsagar)

URL: #15287
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working non-breaking Non-breaking change Python Affects Python cuDF API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] groupby(...).apply raises when aggregation returns multiple outputs
3 participants