search

desihub / desispec

DESI spectral pipeline
BSD 3-Clause "New" or "Revised" License
35 stars 24 forks source link

desi_group_spectra drops SURVEY, PROGRAM info #2170

Closed sbailey closed 5 months ago

sbailey commented 7 months ago

desi_group_spectra drops SURVEY and PROGRAM header keywords from the FIBERMAP HDU when combining cframe files into spectra files. In general, desi_group_spectra can combine across tiles and thus this could be ambiguous, but when the --onetile option is specified it knows it is combining a single tile with a unique (SURVEY, PROGRAM), which would be useful to propagate into the spectra files (and thus downstream into the coadd and redrock files too).

sbailey commented 5 months ago

Fixed in PR #2176. Re-verified with

[login20] desi_group_spectra --onetile --inframes \
  /global/cfs/cdirs/desi/spectro/redux/iron/exposures/20210517/00089031/cframe-r0-00089031.fits.gz \
  /global/cfs/cdirs/desi/spectro/redux/iron/exposures/20210517/00089031/cframe-z0-00089031.fits.gz \
  /global/cfs/cdirs/desi/spectro/redux/iron/exposures/20210517/00089031/cframe-b0-00089031.fits.gz \
  --outfile $SCRATCH/temp/spectra-test.fits --coaddfile $SCRATCH/temp/coadd-test.fits
...

[login20] fitsheader $SCRATCH/temp/spectra-test.fits -e FIBERMAP -k SURVEY -k PROGRAM
# HDU FIBERMAP in /pscratch/sd/s/sjbailey/temp/spectra-test.fits:
SURVEY  = 'main    '                                                            
PROGRAM = 'DARK    '                                                            

[login20] fitsheader $SCRATCH/temp/coadd-test.fits -e FIBERMAP -k SURVEY -k PROGRAM
# HDU FIBERMAP in /pscratch/sd/s/sjbailey/temp/coadd-test.fits:
SURVEY  = 'main    '                                                            
PROGRAM = 'DARK    '