-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DEPR: pd.Grouper key -> on #15331
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
Comments
@chrisaycock gives +1 |
|
any thoughts about whether we should do this (at some point)? |
No strong opinion. |
Any clarity on this? |
Since it's only about a possible future deprecation (not a deprecation that can be enforced in 1.0), there is not need to have this in 1.0 (we can deprecate things in any 1.x afterwards) and so I moved this off the 1.0 milestone. |
Since we decided to keep Grouper (#41297) and there is no strong leaning on this, probably best to keep it as necessary to the Grouper API. Closing but can reopen if we redecide. |
xref #15326 (comment)
pd.Grouper
was introduced, we usedkey
to indicate the column which to group on.resample
added the same idea ason
on
inpd.merge
(existing for a long time).groupby
usesby
(though to be fair this can take lots of things and is not strictly a string column name).So the proposal is to deprecate
pd.Grouper(key=...)
in favor ofon=
.The text was updated successfully, but these errors were encountered: