-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
DEPR: Enforce numeric_only=False in groupby sum/mean #49829
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
DEPR: Enforce numeric_only=False in groupby sum/mean #49829
Conversation
is the failed asv time_pivot_table_margins_only_column a false-positive? |
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.
Generally looks good, just got minor comments / a question
Thanks @jbrockmendel - ASV failure was due to trying to take the mean of a string column; I added it to the |
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 cc @MarcoGorelli merge when ready
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.
looks good to me, thanks @rhshadrach !
* DEPR: Enforce numeric_only=False in groupby sum/mean * cleanup * Refinements * whatsnew fixup
doc/source/whatsnew/vX.X.X.rst
file if fixing a bug or adding a new feature.Targeting these two ops first because they're used as an example op in a lot of tests. Hoping the rest of the ops are easier. Note this doesn't check off the corresponding line in the list of deprecations.