-
Notifications
You must be signed in to change notification settings - Fork 285
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
Delete equalise_cubes.py and add whatsnew #4496
Delete equalise_cubes.py and add whatsnew #4496
Conversation
Not sure where in what's new this belongs - is it Internal or a Deprecation (which I assumed were things we're deprecating now, not removing having previously deprecated them) |
I'm tempted to say it may make more sense for the whats new to be added to deprecations. We state that something will be deprecated, and then follow this up by confirming it has actually been removed, and no longer available. As a user, if I wanted to check when it was actually removed, my first port of call would be to look under the "deprecations" section as it is related to that. |
Don't think I made myself so clear in the lat comment, so to follow up: I think removing code is part of the deprecation process so the what's new should live under the deprecations heading |
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 agree it makes sense to have the whatsnew in internal. The user will see no functional change (other than a different error message if they try to import this module).
Edit: x-post with @lbdreyer and will defer to her!
Whoops! danger of commenting at the same time! But I suppose this proves there's no correct answer; both have valid reasons. Another point I wanted to mention though: I think this is a bit of a strange deprecation, in that we changed the code of the old function. Normally I would expect, when using the function, to get a deprecation warning, but to still be able to use it, until its eventual removal. In that example, I would say it definitely belongs in deprecations |
Thanks @wjbenfold ! |
🚀 Pull Request
Description
Remove
iris.experimental.equalise_cube
, closing #3528Consult Iris pull request check list