specutils icon indicating copy to clipboard operation
specutils copied to clipboard

identify-tabular-fits more strict JWST test

Open kelle opened this issue 7 months ago • 3 comments

Currently, tabular-fits won't identify on any spectrum with TELESCOP=JWST. This is far too broad since folks are re-reducing JWST data and outputting tabular-fits format spectra. I've modified the identifier to only ignore JWST spectra with specific HDU names that have their own JWST loaders.

kelle avatar May 02 '25 20:05 kelle

I will also write some tests with a JWST spectrum which meets these new criteria.

kelle avatar May 03 '25 17:05 kelle

I'm not quite sure where to write the test. In the meantime, here's a test spectrum which should be auto-identified by tabular-fits: CWISEPJ1446-23NIR.fits.zip

kelle avatar May 07 '25 23:05 kelle

Edits made based on review! please take another look.

kelle avatar Jun 26 '25 22:06 kelle

Merge please? 🥹

kelle avatar Jul 18 '25 20:07 kelle