search

USRA-STI / gdt-fermi

Gamma-ray Data Tools - Fermi mission components
Apache License 2.0
2 stars 3 forks source link

Copy TTE header info when converting to PHAII #6

Closed AdamGoldstein-USRA closed 1 year ago

AdamGoldstein-USRA commented 1 year ago

Resolved a TODO and bug by copying relevant header information. The problem was discovered when the detector name was expected to be copied over and it was not. The issue was solved by copying all relevant header information, including detector name, to the PHAII header on conversion. Updated unit tests.

AdamGoldstein-USRA commented 1 year ago

Yes, I intended PHAII. CTIME and CSPEC are defined by their time and energy binnings, and an arbitrary binning created from TTE won't necessarily match the CTIME and CSPEC definitions. Someone who wants to make a true CTIME or CSPEC file from TTE can always update that keyword.