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

Add correction of systematic photodiode error

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: cp_pipe, ip_isr
    • Labels:
      None
    • Team:
      External

      Description

      When using the BOT photodiode data to create the amp linearizers, there is a systematic error in the photodiode data that increases the residuals of the linearizer fit. Aaron Roodman has pointed out that the residuals are highly correlated across the focal plane, so by taking the mean (or median) of the residuals of all of the amps, we can create a correction to the photodiode reading that reduces the residuals. The attached plot shows that the reduction in the residuals is pretty dramatic. This ticket is for implementing that code within the existing framework.

        Attachments

          Issue Links

            Activity

            Hide
            czw Christopher Waters added a comment -

            Two small remaining style issues, but the code looks good.

            Show
            czw Christopher Waters added a comment - Two small remaining style issues, but the code looks good.
            Hide
            mfisherlevine Merlin Fisher-Levine added a comment -

            I've fixed the remaining issues. Jenkins is running, I'll merge on completion.

            Show
            mfisherlevine Merlin Fisher-Levine added a comment - I've fixed the remaining issues. Jenkins is running, I'll merge on completion.
            Hide
            mfisherlevine Merlin Fisher-Levine added a comment -

            Jenkins found errors in obs_decam, Craig fixed these, Jenkins then passed, and I've now merged.

            Show
            mfisherlevine Merlin Fisher-Levine added a comment - Jenkins found errors in obs_decam, Craig fixed these, Jenkins then passed, and I've now merged.

              People

              Assignee:
              cslage Craig Lage
              Reporter:
              cslage Craig Lage
              Reviewers:
              Andrés Alejandro Plazas Malagón, Christopher Waters
              Watchers:
              Andrés Alejandro Plazas Malagón, Christopher Waters, Craig Lage, Merlin Fisher-Levine
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.