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
Remove deprecated code from wcs
#15504
Conversation
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
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.
Needs a change log. Thanks!
Still need a change log. |
I'd first like to know what should be done with |
This? Yeah, not sure. Looks like the warning is always emitted if this gets called. Maybe @mcara would know. But this can also be a follow-up PR because I feel like bigger changes are needed to get rid of this warning. def _read_d2im_old_format(self, header, fobj, axiscorr):
warnings.warn(
"The use of ``AXISCORR`` for D2IM correction has been"
" deprecated.`~astropy.wcs` will read in files with ``AXISCORR`` but"
" ``to_fits()`` will write out files without it.",
AstropyDeprecationWarning,
) |
To elaborate: from the deprecation warning in |
I haven't heard anything about |
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.
LGTM now. Thanks!
Description
The first commit removes code deprecated in #10229. The users are meant to import things from
astropy.wcs.wcsapi
and apparently the internals of that module are considered private. The deprecation therefore had no change log entry, and the removal of the code shouldn't have an entry either.The second commit removes code deprecated in #6682. There were no change log entries announcing the deprecation, so likewise there's no entry announcing the removal.
There is one more deprecated function:
astropy/astropy/wcs/wcs.py
Lines 966 to 972 in 98d82dd
But in this case I'm not sure if removing the deprecated function is the right thing to do. Maybe the warning type should be changed instead. I'd prefer if someone more familiar with
wcs
would have a look at this.