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

FITS header sanity #12

Open
teuben opened this issue Jan 10, 2021 · 3 comments
Open

FITS header sanity #12

teuben opened this issue Jan 10, 2021 · 3 comments

Comments

@teuben
Copy link
Contributor

teuben commented Jan 10, 2021

Various items in the cube FITS header need to be checked with 3rd party packages to see if we can switch between VLSR and FREQ for example. The latest version (10-jan-2021) now reports the history of how the dataset was created.
There are still some comments in the fits comment field where we need to pay attention (e.g. SPECSYS, RADECSYS). The issue around how the VLSR is supposed to be written now seems to be ZSOURCE, not VLSR or VSOURCE.

The only non-standard keyword we write is OBSNUM (even for a combination, which is arguably wrong)

@teuben
Copy link
Contributor Author

teuben commented Jan 10, 2021

As an example, for the benchmark IRC data, in the fits file we have 350.303 km/s at the reference pixel, but once in CASA, this becomes 349.894. cdelt3 is 1.015921 km/s, but in CASA it's 1.013554. Perhaps we need to supply ALTRVAL/ALTRPIX?

@teuben
Copy link
Contributor Author

teuben commented Jan 11, 2021

It seems CASA is using VRAD, not VELO-LSR, on export. The circle with miriad is not rounded yet though.

@teuben
Copy link
Contributor Author

teuben commented Mar 7, 2021

aplpy still has an issue with a cube. claims not a proper wcs.

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

No branches or pull requests

1 participant