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
Ensure tables with masked and unmasked columns roundtrip properly #15473
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.
I'm not a FITS maintainer but this looks correct as far as I can tell. The change seems confined to the narrow context of the masking problem for int columns and the test demonstrates that the fix works.
Thanks for the review! I think this is trivial enough I can just merge (after all, I'm just correcting bugs in stuff I wrote...) |
This comment was marked as resolved.
This comment was marked as resolved.
…umns roundtrip properly
…umns roundtrip properly
…473-on-v5.3.x Backport PR #15473 on branch v5.3.x (Ensure tables with masked and unmasked columns roundtrip properly)
Backport PR #15473: Ensure tables with masked and unmasked columns roundtrip properly (v5.0.x)
Description
This pull request is to address that when a table was written to FITS and read back, all integer columns would become masked if any column in the table was masked.
Fixes #15417