| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
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.
|
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Description
FITS binary tables are limited to 999 columns. Writing a larger Table currently reaches HDU construction before that limit is reported, which can emit non-standard TTYPE1000/TFORM1000 warnings before ending with the generic message that TFIELDS is invalid.
This change checks the encoded table's physical column count before constructing the HDU and raises a specific VerifyError. The check happens after mixin encoding so expanded mixin columns are included.
The regression tests cover both sides of the boundary: 999 columns still write successfully, while 1000 columns raise the clear error.
Fixes #19236
Validation
AI disclosure
OpenAI Codex assisted with issue triage, implementation, and local validation. The contributor remains responsible for the submitted content and will engage with review feedback.