Skip to content

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

Closed
jreback opened this issue Feb 7, 2017 · 7 comments
Closed

DEPR: pd.Grouper key -> on #15331

jreback opened this issue Feb 7, 2017 · 7 comments
Labels
Deprecate Functionality to remove in pandas Groupby

Comments

@jreback
Copy link
Contributor

jreback commented Feb 7, 2017

xref #15326 (comment)

  • when pd.Grouper was introduced, we used key to indicate the column which to group on
  • .resample added the same idea as on
  • we use on in pd.merge (existing for a long time)
  • .groupby uses by (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 of on=.

@jreback jreback added API Design Deprecate Functionality to remove in pandas labels Feb 7, 2017
@jreback jreback added this to the 0.20.0 milestone Feb 7, 2017
@jreback
Copy link
Contributor Author

jreback commented Feb 7, 2017

@chrisaycock gives +1
@jorisvandenbossche -0
@jreback +1

@chrisaycock
Copy link
Contributor

pd.merge_asof() has both on and by parameters.

@jreback jreback modified the milestones: 0.20.0, 0.21.0 Mar 23, 2017
@jreback
Copy link
Contributor Author

jreback commented Oct 2, 2017

any thoughts about whether we should do this (at some point)?

@TomAugspurger @jorisvandenbossche

@jreback jreback modified the milestones: 0.21.0, 1.0 Oct 2, 2017
@TomAugspurger
Copy link
Contributor

No strong opinion.

@jbrockmendel
Copy link
Member

Any clarity on this?

@jorisvandenbossche jorisvandenbossche modified the milestones: 1.0, 1.1 Dec 11, 2019
@jorisvandenbossche
Copy link
Member

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.

@TomAugspurger TomAugspurger modified the milestones: 1.1, Contributions Welcome Jul 6, 2020
@mroeschke
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Deprecate Functionality to remove in pandas Groupby
Projects
None yet
Development

No branches or pull requests

6 participants