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

use keyword-only params in sort_index #188

Merged
merged 1 commit into from
Jul 3, 2022

Conversation

maurerle
Copy link
Contributor

in methods where inplace is allowed, non-keyword args will become deprecated soon:
pandas-dev/pandas#41506

This does not change any current behavior, but removes a deprecation warning when using the latest pandas version

in methods where inplace is allowed, non-keyword args will become deprecated soon:
pandas-dev/pandas#41506

This does not change any current behavior, but removes a deprecation warning when using the latest pandas version
@fboerman
Copy link
Collaborator

fboerman commented Jul 3, 2022

hi @maurerle thank you, is that really the only place where this is a problem?

@maurerle
Copy link
Contributor Author

maurerle commented Jul 3, 2022

I heavily used this library in the past days with python 3.10 and latest pandas and this was the only occurence where I saw the DeprecationWarning from pandas.
Furthermore I looked through the code a little bit and didn't see it elsewhere.

One could also directly remove the usage of inplace to be safe for the next few years - as this is the source of the issue, but for now there are no deprecationwarnings for this.

@fboerman
Copy link
Collaborator

fboerman commented Jul 3, 2022

hi @maurerle thanks then ill merge this!

@fboerman fboerman merged commit 7b451d5 into EnergieID:master Jul 3, 2022
fleimgruber pushed a commit to fleimgruber/entsoe-py that referenced this pull request May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants