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: numeric_only=None in NDFrame.rank #45036

Closed
jbrockmendel opened this issue Dec 23, 2021 · 5 comments · Fixed by #45059
Closed

DEPR: numeric_only=None in NDFrame.rank #45036

jbrockmendel opened this issue Dec 23, 2021 · 5 comments · Fixed by #45059
Labels
Bug Needs Triage Issue that has not been reviewed by a pandas team member
Milestone

Comments

@jbrockmendel
Copy link
Member

jbrockmendel commented Dec 23, 2021

We've deprecated it in DataFrame and groupby reductions, would be nice to get it here too.

@jbrockmendel jbrockmendel added Bug Needs Triage Issue that has not been reviewed by a pandas team member labels Dec 23, 2021
@phofl
Copy link
Member

phofl commented Dec 24, 2021

Are you sure? In groupby numeric_only is not accepted and in DataFrame.rank this does not show a warning

@jbrockmendel
Copy link
Member Author

in DataFrame.rank this does not show a warning

I'm suggesting we should deprecated so that it does show a warning.

        # if numeric_only is None, and we can't get anything, we try with
        # numeric_only=True
        if numeric_only is None:
            try:
                return ranker(self)
            except TypeError:
                numeric_only = True

@phofl
Copy link
Member

phofl commented Dec 24, 2021

Ok sorry, misunderstood your initial comment

@jreback jreback added this to the 1.4 milestone Dec 27, 2021
@jreback
Copy link
Contributor

jreback commented Dec 27, 2021

I would ideally like to kill numeric_only everywhere but i guess its convenient (rant)

@jbrockmendel
Copy link
Member Author

I would ideally like to kill numeric_only everywhere but i guess its convenient (rant)

we've mostly gotten rid of numeric_only=None, which is the biggest hassle

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Needs Triage Issue that has not been reviewed by a pandas team member
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants