desi_extract_spectra: allow MASK and READNOISE to be optional
- Dominant language
- Python
- Stars
- 42
- Forks
- 26
- Avg merge
- 1d 21h
- Merged PRs (30d)
- 15
Description
This came up while trying to apply DESI code to BOSS data: it would be handy to make the MASK and READNOISE extensions to be optional for desi_extract_spectra. The mask would default to 0 (nothing masked, allowing IVAR=0 to capture anything that should be masked), and a missing READNOISE HDU would fall back to looking for RDNOISE keywords (possibly RDNOISEn for different amps).
Contributor guide
No contributing guide indexed for this repository
Research direction
Start at the desi_extract_spectra entry point and inspect how the MASK and READNOISE extensions are currently required. Implement the requested optional behavior, including the mask default and RDNOISE keyword fallback, then run the existing extraction tests or relevant pipeline checks to confirm BOSS-style inputs work.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100