-
-
Notifications
You must be signed in to change notification settings - Fork 18.1k
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: Deprecate NDFrame.filter #27617
Conversation
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.
ok, we need to decide on #27363 first
@topper-123 can you rebase |
55acd2c
to
36cbcf0
Compare
can you rebase again. |
36cbcf0
to
da9b64e
Compare
I've rebased. As you noted, this depends on #27363. |
@topper-123 im trying to clear the PR backlog; can this be closed pending #27363? |
Knowing the outstanding issue is around regex capability, maybe we should just deprecate all keywords from filter for now that aren't regex? |
That sounds like a stepping stone to deprecating. Wouldn't it make more sense then to just deprecate this method before 1.0 is released? |
I don't think we'd want to outright deprecate without a replacement for the regex unless we instruct users to do |
Oh yes, I meant choose another way to select by regex at the same time, sorry. |
can you merge master and will look again |
da9b64e
to
617e231
Compare
I've rebased. I do think an alternative should be settled on for the regex use case and I've had two proposals that were not accepted. So I'm not sure where this is floating currently? |
Thanks for the PR! But I think agreement on approach here is still outstanding. If something you want to drive forward can revisit discussion in issue first |
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff
I propose to deprecate (DataFrame|Series).filter. The method's functionality is already in the
.loc
method, except the regex functionality, for which I've made a proposal adding regex functionality to.loc
in #27363.