Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-2640

Update AssembleCcdTask to handle missing amps

    XMLWordPrintable

    Details

    • Type: Story
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: ip_isr
    • Labels:
      None
    • Team:
      Alert Production

      Description

      Think about what AssembleCcdTask should do when passed incomplete AmpInfo tables. Options are:

      • Raise an exception
      • Fill gaps with NaNs (which subsequently get interpolated over by IsrTask)

      Considerations:

      • Amps will fail

        Attachments

          Issue Links

          mentioned in

          Page Loading...

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            yusra Yusra AlSayyad created issue -
            Hide
            Permalink
            rowen Russell Owen added a comment -

            I don't think a failed amp can reasonably be handled by an incomplete AmpInfo table. Rather the AmpInfo table needs a flag indicating that an amplifier is bad. If that flag is set then some other data in the table should be ignored. But the bounding boxes indicating raw and assembled data are still relevant. Thus I don't think it make sense to support incomplete AmpInfo tables.

            Show
            rowen Russell Owen added a comment - I don't think a failed amp can reasonably be handled by an incomplete AmpInfo table. Rather the AmpInfo table needs a flag indicating that an amplifier is bad. If that flag is set then some other data in the table should be ignored. But the bounding boxes indicating raw and assembled data are still relevant. Thus I don't think it make sense to support incomplete AmpInfo tables.
            jbecla Jacek Becla made changes -
            Field Original Value New Value
            Team Alert Production [ 10300 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 21262 ]
            Hide
            Permalink
            swinbank John Swinbank added a comment -

            Discussed at the DM-CCB of 2019-08-21.

            Show
            swinbank John Swinbank added a comment - Discussed at the DM-CCB of 2019-08-21.
            tjenness Tim Jenness made changes -
            Remote Link This issue links to "Page (Confluence)" [ 36914 ]
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Christopher Waters do you have any thoughts on whether this ticket is still relevant?

            Show
            tjenness Tim Jenness added a comment - Christopher Waters do you have any thoughts on whether this ticket is still relevant?

              People

              Assignee:
              yusra Yusra AlSayyad
              Reporter:
              yusra Yusra AlSayyad
              Watchers:
              John Swinbank, Russell Owen, Simon Krughoff, Tim Jenness, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:

                  Jenkins

                  No builds found.