Skip to content
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: Series.nlargest/nsmallest take_last. #10920

Merged
merged 1 commit into from
Aug 29, 2015
Merged

Conversation

sinhrks
Copy link
Member

@sinhrks sinhrks commented Aug 28, 2015

Closes #10792.

Because using deprecate_kwargs hides original impl, used ugly workaround for GroupBy...

@sinhrks sinhrks added Reshaping Concat, Merge/Join, Stack/Unstack, Explode Deprecate Functionality to remove in pandas labels Aug 28, 2015
@sinhrks sinhrks added this to the 0.17.0 milestone Aug 28, 2015
if f.__name__ in ['nsmallest', 'nlargest']:
if len(args) == 0:
decl = ['self', 'n=5', "keep='first'"]
args = ['self', 'n', 'keep']
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't do this
simply define the function and treat normally (eg see irow)

@sinhrks sinhrks force-pushed the nlargest branch 2 times, most recently from 2fb8bdc to 3a8d1d6 Compare August 28, 2015 21:59
@sinhrks
Copy link
Member Author

sinhrks commented Aug 29, 2015

@jreback Yes, updated.

jreback added a commit that referenced this pull request Aug 29, 2015
DEPR: Series.nlargest/nsmallest take_last.
@jreback jreback merged commit 66174e5 into pandas-dev:master Aug 29, 2015
@jreback
Copy link
Contributor

jreback commented Aug 29, 2015

thank you sir!

@sinhrks sinhrks deleted the nlargest branch August 29, 2015 00:10
gfyoung added a commit to forking-repos/pandas that referenced this pull request Mar 16, 2017
Affected methods:

1) nlargest
2) nsmallest
3) duplicated
4) drop_duplicates

xref pandas-devgh-10236, pandas-devgh-10792, pandas-devgh-10920.
gfyoung added a commit to forking-repos/pandas that referenced this pull request Mar 16, 2017
Affected methods:

1) nlargest
2) nsmallest
3) duplicated
4) drop_duplicates

xref pandas-devgh-10236, pandas-devgh-10792, pandas-devgh-10920.
gfyoung added a commit to forking-repos/pandas that referenced this pull request Mar 17, 2017
Affected methods:

1) nlargest
2) nsmallest
3) duplicated
4) drop_duplicates

xref pandas-devgh-10236, pandas-devgh-10792, pandas-devgh-10920.
gfyoung added a commit to forking-repos/pandas that referenced this pull request Mar 17, 2017
Affected methods:

1) nlargest
2) nsmallest
3) duplicated
4) drop_duplicates

xref pandas-devgh-10236, pandas-devgh-10792, pandas-devgh-10920.
gfyoung added a commit to forking-repos/pandas that referenced this pull request Mar 17, 2017
Affected methods:

1) nlargest
2) nsmallest
3) duplicated
4) drop_duplicates

xref pandas-devgh-10236, pandas-devgh-10792, pandas-devgh-10920.
gfyoung added a commit to forking-repos/pandas that referenced this pull request Mar 17, 2017
Affected methods:

1) nlargest
2) nsmallest
3) duplicated
4) drop_duplicates

xref pandas-devgh-10236, pandas-devgh-10792, pandas-devgh-10920.
gfyoung added a commit to forking-repos/pandas that referenced this pull request Mar 17, 2017
Affected methods:

1) nlargest
2) nsmallest
3) duplicated
4) drop_duplicates

xref pandas-devgh-10236, pandas-devgh-10792, pandas-devgh-10920.
gfyoung added a commit to forking-repos/pandas that referenced this pull request Mar 17, 2017
Affected methods:

1) nlargest
2) nsmallest
3) duplicated
4) drop_duplicates

xref pandas-devgh-10236, pandas-devgh-10792, pandas-devgh-10920.
gfyoung added a commit to forking-repos/pandas that referenced this pull request Mar 17, 2017
Affected methods:

1) nlargest
2) nsmallest
3) duplicated
4) drop_duplicates

xref pandas-devgh-10236, pandas-devgh-10792, pandas-devgh-10920.
jreback pushed a commit that referenced this pull request Mar 18, 2017
Affected methods:

1) nlargest
2) nsmallest
3) duplicated
4) drop_duplicates

xref #10236, #10792, #10920.

Author: gfyoung <[email protected]>

Closes #15710 from gfyoung/create-last-kw-drop and squashes the following commits:

b416290 [gfyoung] MAINT: Drop take_last kwarg from method signatures
AnkurDedania pushed a commit to AnkurDedania/pandas that referenced this pull request Mar 21, 2017
Affected methods:

1) nlargest
2) nsmallest
3) duplicated
4) drop_duplicates

xref pandas-dev#10236, pandas-dev#10792, pandas-dev#10920.

Author: gfyoung <[email protected]>

Closes pandas-dev#15710 from gfyoung/create-last-kw-drop and squashes the following commits:

b416290 [gfyoung] MAINT: Drop take_last kwarg from method signatures
mattip pushed a commit to mattip/pandas that referenced this pull request Apr 3, 2017
Affected methods:

1) nlargest
2) nsmallest
3) duplicated
4) drop_duplicates

xref pandas-dev#10236, pandas-dev#10792, pandas-dev#10920.

Author: gfyoung <[email protected]>

Closes pandas-dev#15710 from gfyoung/create-last-kw-drop and squashes the following commits:

b416290 [gfyoung] MAINT: Drop take_last kwarg from method signatures
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 Reshaping Concat, Merge/Join, Stack/Unstack, Explode
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants