IGRINSSpectrumList Note

[ericasaw]

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]

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.

[kfkaplan]

@ericasaw Is this fixed now that add_flux_calibration has been merged with main? If so, I think we can close the issue.