OttoStruve/muler

IGRINSSpectrumList Note

ericasaw opened this issue · 1 comments

Just a reminder for us later, IGRINSSpectrumList in the current main version of muler v.0.4.0 doesnt allow you to pass in an array of fluxes and wavelengths to construct an IGRINSSpectrumList object. This should be fixed in the newest add_flux_calibration branch--but just a note for us later

gully commented

On tradeoff with this directly-populate approach is that the IGRINSSpectrumList will lack metadata associated with the spectrum (such as time stamps, RA, and Dec). So certain opertations, like barycentric_correct won't work. That's fine, we'll simply want to anticipate such scenarios by raising informative error messages when users attempt to call IGRINSSpectrumList when such metadata is not present.