-
Notifications
You must be signed in to change notification settings - Fork 27
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
What to do if OGIP conventions and the FITS standard are in conflict? #191
Comments
Maybe check what other instruments do as this at the end corresponds most to what users (and software) expects.
… Le 24 févr. 2022 à 10:53, Maximilian Nöthe ***@***.***> a écrit :
I just noticed a conflict between the OGIP conventions and the FITS standard, which is explicitly commented on in the FITS time paper, which definitions were taken over for the FITS standard version 4.0.
Specifically, in this instance, it is the keyword and its possible values specifying the time reference location.
In OGIP, it is TIMEREF with value 'LOCAL', which is given in GADF (but, pointing incorrectly to the FITS standard) and in the FITS time paper and the current standard it is TREFPOS with multiple possible values and 'TOPOCENTER' being the equivalent to 'LOCAL'.
While reworking the time definitions is also needed with respect to the current standard, I think we should take a decision on how to resolve conflicts between FITS standard and OGIP convention.
My personal preference would be to follow the FITS standard.
—
Reply to this email directly, view it on GitHub <#191>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAW2QV3Z4MA5QKEUDE7JWVTU4X5ZFANCNFSM5PG4CI5Q>.
Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you are subscribed to this thread.
|
I don't think we should let the legacy behavior of other instruments or software take precedence over the current version of the FITS standard for new iterations of the specification. FERMI for example uses I think we should take the decision to give the FITS standard precedence in these kinds of cases, so we are in agreement with the current version and not perpetuate legacy behavior. EDIT: FERMI references the FITS standard Version 2.0 from 2001 in their comment section of photon files freshly downloaded from their server for observation dates in 2021. |
Historically, FITS was developed to interchange data on tape, mostly from the Radio and space community (from IUE to CGRO). The adopted convention is to make all changes such that they do not break the older FITS versions. As a result, there were extensions to the FITS from both OGIP and CDS which differed. It is allowed to differ, for example, the Swift headers specify time in a way that is discouraged nowadays, but in 2004 was OK. So I would not worry too much about the above. As long as you don't break the backwards compatibility it is OK. |
I just noticed a conflict between the OGIP conventions and the FITS standard, which is explicitly commented on in the FITS time paper, which definitions were taken over for the FITS standard version 4.0.
Specifically, in this instance, it is the keyword and its possible values specifying the time reference location.
In OGIP, it is
TIMEREF
with value'LOCAL'
, which is given in GADF (but, pointing incorrectly to the FITS standard) and in the FITS time paper and the current standard it isTREFPOS
with multiple possible values and'TOPOCENTER'
being the equivalent to'LOCAL'
.While reworking the time definitions is also needed with respect to the current standard, I think we should take a decision on how to resolve conflicts between FITS standard and OGIP convention.
My personal preference would be to follow the FITS standard.
From the FITS time paper:
FITS Time paper: https://www.aanda.org/articles/aa/pdf/2015/02/aa24653-14.pdf
FITS Standard Section 9.2.3 (TREFPOS): https://fits.gsfc.nasa.gov/standard40/fits_standard40aa-le.pdf
OGIP definition: https://heasarc.gsfc.nasa.gov/docs/heasarc/ofwg/docs/rates/ogip_93_003/ogip_93_003.html
The text was updated successfully, but these errors were encountered: