-
-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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: Update groupby.apply DeprecationWarning to FutureWarning #59751
DEPR: Update groupby.apply DeprecationWarning to FutureWarning #59751
Conversation
Just got caught up with some of the chat on slack, marking this as a draft until #59513 is merged. |
This pull request is stale because it has been open for thirty days with no activity. Please update and respond to this comment if you're still interested in working on this. |
…upby_apply_future_warning
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'll admit I didn't track the full scope of this, but as far as the change from Deprecation to FutureWarning goes in 2.3 I would be on board.
I see this has been open for a while. Let's give @jorisvandenbossche a few days in case of any objection; otherwise can merge!
I also assume we discussed the actual deprecation thoroughly before, and then +1 on updating it from DeprecationWarning to FutureWarning now. |
@rhshadrach can you add a brief whatsnew note for 2.3 (maybe just linking to the earlier whatsnew note about it). Users might still start seeing this when updating to 2.3, so it would be good to have some mention of it. |
…upby_apply_future_warning
…hadrach/pandas into groupby_apply_future_warning
Thanks @rhshadrach |
doc/source/whatsnew/vX.X.X.rst
file if fixing a bug or adding a new feature.Ref: #54950 (comment)