-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
DEP: update version in deprecation warning of interpnd #22099
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
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.
Thanks for tackling this. If it's urgent we can merge this as-is, though I'd like to fix this in a way that we don't forget it again so easily, see suggestion below.
btw, there's a similar deprecation message issue in e.g. scipy/scipy/_lib/deprecation.py Lines 220 to 222 in ba8553c
(it's the missing which leads to e.g.
(i.e. the |
oh and the |
Great spot thanks, has now been fixed.
@steppi what's the plan here? |
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.
I'd like to understand better why we have so many warnings that (before this PR) say that stuff will be removed in 1.14. That means either the warnings were all are wrong, or we missed to remove a lot of stuff.
Sure, but I'm not clear which version I should mention that it'll be removed at. |
Look at when the deprecation in the docstring was added (1.15 IIUC?), then add two minor versions (=1.17) |
Ah ok, I wasn't aware that the docstring was added in 1.15, because it looked like it was just forgotten to be removed 🤷🏻 |
Thanks for the review @h-vetinari and all the eagle eyed spots @jorenham |
Closes #22097
It was reported that
interpolate.interpnd.GradientEstimationWarning
etc raised a warning saying they would be removed in 1.14. In fact they were only deprecated in this cycle and the version in the warning was outdated. This pr updates the version number so they can be removed in the next release.