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

Set SUSPECT mask in ISR task and make saturation a double

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ip_isr
    • Story Points:
      5
    • Epic Link:
    • Sprint:
      Alert Production X16 - 5
    • Team:
      Alert Production

      Description

      Implement RFC-190 and mask suspect pixels:

      Add selectLevel to lsst.afw.cameraGeom.AmpInfoCatalog, as a double, and add support for it to lsst.ip.isr.IsrTask, analogously to masking saturation: iif suspectLevel is not nan then set the SUSPECT flag for pixels above the suspect level.

      Also change the type of saturation in the AmpInfoCatalog from int to double, so that the existing test for nan actually works

        Attachments

          Issue Links

          blocks

          Improvement - An improvement or enhancement to an existing feature or task. DM-5462 Add non-linearity correction to ISR task

          • Done
          is triggered by

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-190 Use nan for saturation level to prevent masking

          • Implemented
          relates to

          Bug - A problem which impairs or prevents the functions of the product. DM-6246 Vertical overscan off by one again

          • Done

          Improvement - An improvement or enhancement to an existing feature or task. DM-6161 Add unit test for IsrTask

          • Invalid

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-192 Single source for camera data

          • Adopted

            Activity

            Ascending order - Click to sort in descending order
            rowen Russell Owen created issue -
            rowen Russell Owen made changes -
            Field Original Value New Value
            Link This issue is triggered by RFC-190 [ RFC-190 ]
            rowen Russell Owen made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            rowen Russell Owen made changes -
            Link This issue blocks DM-5462 [ DM-5462 ]
            rowen Russell Owen made changes -
            Epic Link DM-3878 [ 20173 ]
            rowen Russell Owen made changes -
            Reviewers Nate Lust [ nlust ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            rowen Russell Owen made changes -
            Summary Set SUSPECT mask in ISR task Set SUSPECT mask in ISR task and make saturation a double
            rowen Russell Owen made changes -
            Reviewers Nate Lust [ nlust ] John Parejko [ parejkoj ]
            rowen Russell Owen made changes -
            Watchers Nate Lust, Russell Owen [ Nate Lust, Russell Owen ] Nate Lust, Paul Price, Russell Owen [ Nate Lust, Paul Price, Russell Owen ]
            rowen Russell Owen made changes -
            Reviewers John Parejko [ parejkoj ] Nate Lust [ nlust ]
            rowen Russell Owen made changes -
            Reviewers Nate Lust [ nlust ] John Parejko, Nate Lust [ parejkoj, nlust ]
            rowen Russell Owen made changes -
            Link This issue relates to DM-6161 [ DM-6161 ]
            Parejkoj John Parejko made changes -
            Reviewers John Parejko, Nate Lust [ parejkoj, nlust ] Nate Lust [ nlust ]
            nlust Nate Lust made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            rowen Russell Owen made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            rowen Russell Owen made changes -
            Link This issue relates to DM-6246 [ DM-6246 ]
            rowen Russell Owen made changes -
            Link This issue relates to RFC-192 [ RFC-192 ]

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Reviewers:
              Nate Lust
              Watchers:
              John Parejko, John Swinbank, Nate Lust, Paul Price, Russell Owen
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.