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

Migrate measureCrosstalk.py to cp_pipe

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: cp_pipe, ip_isr
    • Labels:
      None
    • Story Points:
      20
    • Epic Link:
    • Sprint:
      DRP S20-6 (May), DRP F20-1 (June), DRP F20-2 (July)
    • Team:
      Data Release Production

      Description

      measureCrosstalk has historically been in ip_isr, but more logically belongs in cp_pipe.  This ticket will move it, and ensure it is fully updated to use the new CrosstalkCalib.

        Attachments

          Issue Links

            Activity

            Show
            czw Christopher Waters added a comment - Jenkins:  https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/32182/pipeline
            Hide
            czw Christopher Waters added a comment -

            This updates the measureCrosstalk code to be functional with both gen2 and gen3 butlers, fully use the new (lightly updated) CrosstalkCalib, provides preliminary support for inter-chip crosstalk measurements (gen3 only; dependent on support for filtering ConnectionType.Input objects), and saves the flux information that will be needed to support non-linear crosstalk corrections.

            Show
            czw Christopher Waters added a comment - This updates the measureCrosstalk code to be functional with both gen2 and gen3 butlers, fully use the new (lightly updated) CrosstalkCalib, provides preliminary support for inter-chip crosstalk measurements (gen3 only; dependent on support for filtering ConnectionType.Input objects), and saves the flux information that will be needed to support non-linear crosstalk corrections.
            Hide
            mfisherlevine Merlin Fisher-Levine added a comment -

            Generally great, just a lot of comments because there's a lot of code. No major changes required though

            Show
            mfisherlevine Merlin Fisher-Levine added a comment - Generally great, just a lot of comments because there's a lot of code. No major changes required though
            Show
            czw Christopher Waters added a comment - https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/32356/pipeline

              People

              • Assignee:
                czw Christopher Waters
                Reporter:
                czw Christopher Waters
                Reviewers:
                Merlin Fisher-Levine
                Watchers:
                Christopher Waters, Merlin Fisher-Levine
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel