-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Removing deprecated code from io.fits.header #5020
Conversation
…eader.remove: ``ignore_missing`` and ``all``.
keyword = Card.normalize_keyword(keyword) | ||
if keyword in self._keyword_indices: | ||
del self[self._keyword_indices[keyword][0]] | ||
if all: |
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.
all
is maybe not a so good idea as argument name, as it shadows the builtin all
function. Maybe remove_all
?
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.
good point, thanks!
👍 |
Removing deprecated code from io.fits.header
@@ -209,6 +209,9 @@ New Features | |||
- New function ``convenience.table_to_hdu`` to allow creating a FITS | |||
HDU object directly from an astropy ``Table``. [#4778] | |||
|
|||
- A new optional arguments ``ignore_missing`` and ``remove_all`` are added |
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.
Slight typo here. "A new optional arguments".. Should be reworded.
Thanks @bsipocz and @Pratik151 for the original fix. |
This is a rebased ad squashed version of #4445 that was milestoned as 1.2.
I believe @embray gave the green light pending the changelog entry back in Jan, thus it would be nice to get this in the release.
Closes #4445, closes #4429 and most probably #2594, too.
cc @eteq @astrofrog