spacetelescope / jwst

Python library for science observations from the James Webb Space Telescope
https://jwst-pipeline.readthedocs.io/en/latest/
Other
564 stars 167 forks source link

Task: Update datamodels and steps for new integration times table #1881

Closed hbushouse closed 6 years ago

hbushouse commented 6 years ago

SDP is adding a new table extension to all level-1b products that will contain time stamps for each integration within an exposure (see https://jira.stsci.edu/browse/JSDP-387). While primarily intended for use with Time Series exposures, the table will be added to all exposures and is expected to be carried along through all stages of cal pipeline processing.

In order to propagate the table through all stages of pipeline processing we will need to update all relevant science data product data models (e.g. RampModel, ImageModel, SlitModel, etc.) to define this table extension. We will also need to update all calibration steps that create their output(s) as a new datamodel type so that they explicitly copy the integration times table over from the input model to the output model. This includes steps like ramp_fit, extract_2d, cube_build, resample, and resample_spec.

Propagating this table through the various pipeline stages is vital in order to make it available to level-3 TSO steps that need it, like tso_photometry (#1619), extract_1d (#1620), and white_light (#1621).

This is high priority for Build 7.2, in order to satisfy the TSO processing requirement.

hbushouse commented 6 years ago

Chris Heller indicates that each exposure segment will contain integration time table entries that cover at least the integrations contained in that segment, and often a few more at either the beginning or end of the table (because the way science data is downlinked in packets is not always broken on boundaries that match the start or end of an integration). So any pipeline task that needs to use the time stamps will first need to query the INTSTART and INTEND keywords to determine which integrations are contained in the segment being worked on and then use the corresponding entries in the integration times table.

philhodge commented 6 years ago

I'm closing this for the time being, but it may be necessary to re-open later if keywords or column names need to be changed.

hbushouse commented 6 years ago

Given that the DMSWG has decided that the INT_TIMES table does NOT need to be propagated beyond level-1b products for exposures taken in non-TSO modes, we don't need to update nearly as many of the data models or steps to propagate it. We only need to propagate it for TSO exposures. So that means we only need to propagate it through the following steps:

In summary, the following data models need to be updated to define the INT_TIMES table: