-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
REGR: groupby fails with nullable dtypes and dropna=False #48824
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
Conversation
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.
I wouldn't mind the test be split by dtype (numeric, string, date, category) given the dynamic xfails, and change in scalar values and use of x/y/z
to get combos, even if it duplicates the groupby op
Thanks @mroeschke - I was able to simplify this a bit (namely - remove an xfail, less parameters for determining sequence, less repetition of uniques). Also added back in |
Looks better. There's still a bit to grok to understand |
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.
LGTM. Feel free to merge in a few days to see if anyone else has objection with the test complexity
Thanks @rhshadrach |
…s and dropna=False
… dtypes and dropna=False) (#48938) Backport PR #48824: REGR: groupby fails with nullable dtypes and dropna=False Co-authored-by: Richard Shadrach <[email protected]>
…#48824) * REGR: groupby fails with nullable dtypes and dropna=False * Rework test * Merge cleanup * Range fixup
doc/source/whatsnew/vX.X.X.rst
file if fixing a bug or adding a new feature.This reworked test is a bit ghastly, any suggestions on how to simplify or clean up are most welcome.