Skip to content
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

CameraReadout.to_table is not FITS compatible #1452

Closed
kosack opened this issue Oct 9, 2020 · 1 comment
Closed

CameraReadout.to_table is not FITS compatible #1452

kosack opened this issue Oct 9, 2020 · 1 comment

Comments

@kosack
Copy link
Contributor

kosack commented Oct 9, 2020

CameraReadout.to_table().write("readout.fits") creates a HIERARC keyword in the output FITS file, since the REF_WIDTH header is > 8 characters. Suggest changing it to REFWIDTH. Or we just deal with the strange data model (it's fine for other formats like HDF5 and ECSV, just not FITS).

@maxnoe
Copy link
Member

maxnoe commented Oct 9, 2020

Let's use REFWIDTH. I think it was meant to be fits compatible but miscounted, otherwise a more explicit name would have been appropriate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants