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

Exposure.getReadoutCorner method returns incorrect values for some LSST CCD amplifiers.

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ip_isr
    • Labels:
      None
    • Story Points:
      2
    • Epic Link:
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      The `Exposure.getReadoutCorner()` method used to determine the appropriate X/Y flips needed to properly orient two amplifiers to match returns the incorrect value for some amplifiers of the LSST CCD.

      Ex: Both Segment C10 and Segment C00, which are on different sides of the CCD midline break return the same location for readout corner:

      ```

      detector = exp.getDetector()
      amp_top = detector['C10']
      amp_bottom = detector['C00']

      print(amp_top.getReadoutCorner())
      print(amp_bottom.getReadoutCorner())

       

      > ReadoutCorner.LL

      > ReadoutCorner.LL

      ```

      This is an issue for calculating crosstalk between amplifiers on opposite sides of the midline break as the orientation of the two pixel arrays obtained using `lsst.ip.isr.CrosstalkCalib.extractAmp()` is incorrect (see attached images).

        Attachments

        1. amp_comparison_bad.png
          amp_comparison_bad.png
          127 kB
        2. ct-readoutCorners.ipynb
          98 kB
        3. image_example_bad.png
          image_example_bad.png
          528 kB
        4. readout_corners.png
          readout_corners.png
          71 kB

          Activity

          Hide
          czw Christopher Waters added a comment -

          I believe I've tracked this to an issue in afw.  After doing all of the appropriate shifts and flips, the rawFlipX/Y parameters were set to False (as the flips had been done), but the readoutCorner was not.  I've added the appropriate changes to that now, and the attached image shows that the results of CrosstalkCalib.extractAmp now look correct.

          Show
          czw Christopher Waters added a comment - I believe I've tracked this to an issue in afw.  After doing all of the appropriate shifts and flips, the rawFlipX/Y parameters were set to False (as the flips had been done), but the readoutCorner was not.  I've added the appropriate changes to that now, and the attached image shows that the results of CrosstalkCalib.extractAmp now look correct.
          Hide
          czw Christopher Waters added a comment -

          Jenkins success:

          https://ci.lsst.codes/job/stack-os-matrix/35741/display/redirect 

          I've also attached the notebook for my check. ct-readoutCorners.ipynb

          Show
          czw Christopher Waters added a comment - Jenkins success: https://ci.lsst.codes/job/stack-os-matrix/35741/display/redirect   I've also attached the notebook for my check.  ct-readoutCorners.ipynb

            People

            Assignee:
            czw Christopher Waters
            Reporter:
            snyder18 Adam Snyder
            Reviewers:
            Arun Kannawadi
            Watchers:
            Adam Snyder, Arun Kannawadi, Christopher Waters
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.