spacetelescope / ullyses

Code to create products for the ULLYSES program
https://ullyses.stsci.edu/
BSD 3-Clause "New" or "Revised" License
2 stars 6 forks source link

Update SegmentList usage #35

Open jotaylor opened 2 years ago

jotaylor commented 2 years ago

Instead of explicitly calling out STIS and COS, you can use the generic SegmentList since no data is actually being coadded together here. https://github.com/spacetelescope/ullyses/blob/main/ullyses/timeseries.py#L192

jotaylor commented 1 year ago

I believe the original intention here was to abstract away the instrument-specific usage of child SegmentList classes in timeseries. This may be possible, but would require testing. This ticket will be left open for now, but this is an enhancement to generalize the code, not a necessity.