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

calibrations should track sequencer crc

    XMLWordPrintable

Details

    • Improvement
    • Status: Done
    • Resolution: Done
    • None
    • cp_pipe
    • None
    • 4
    • Data Release Production

    Description

      If the sequencer file changes, then we expect that there will be a change in the quality that calibrations created for a previous sequencer will have when applied.  This should be tracked in the files, and if there is a mismatch, we should warn about this.

      Attachments

        Activity

          No builds found.
          czw Christopher Waters created issue -
          yusra Yusra AlSayyad made changes -
          Field Original Value New Value
          Epic Link DM-32156 [ 779836 ] DM-32163 [ 779863 ]
          yusra Yusra AlSayyad made changes -
          Epic Link DM-32163 [ 779863 ] DM-32170 [ 779886 ]
          czw Christopher Waters made changes -
          Status To Do [ 10001 ] In Progress [ 3 ]

          The code changes here only issues log warnings on sequencer mismatch, it does not raise.  

          Jenkins:

          https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/38114/pipeline

          czw Christopher Waters added a comment - The code changes here only issues log warnings on sequencer mismatch, it does not raise.   Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/38114/pipeline
          czw Christopher Waters added a comment - Added config options to handle if this should raise.  New jenkins:  https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/38115/pipeline
          czw Christopher Waters made changes -
          Reviewers Eli Rykoff, Merlin Fisher-Levine [ erykoff, mfisherlevine ]
          Status In Progress [ 3 ] In Review [ 10004 ]

          I'm done with my part of the review. Quite a few comments but they're all very samey, and probably all stem from a basic misunderstanding on my part.

          mfisherlevine Merlin Fisher-Levine added a comment - I'm done with my part of the review. Quite a few comments but they're all very samey, and probably all stem from a basic misunderstanding on my part.
          mfisherlevine Merlin Fisher-Levine made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]

          https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/38138/pipeline/ is the working Jenkins run.  Are we all good on this, or does anyone want to do another iteration of PR comments?

          czw Christopher Waters added a comment - https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/38138/pipeline/  is the working Jenkins run.  Are we all good on this, or does anyone want to do another iteration of PR comments?

          Sorry for being slow, just left some replies to your replies on the PR. Do whatever you like with them though. The only bit I really wonder about is the debug level log, but maybe I'm reading the code/thinking about it wrong.

          mfisherlevine Merlin Fisher-Levine added a comment - Sorry for being slow, just left some replies to your replies on the PR. Do whatever you like with them though. The only bit I really wonder about is the debug level log, but maybe I'm reading the code/thinking about it wrong.
          czw Christopher Waters made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]
          yusra Yusra AlSayyad made changes -
          Story Points 4

          People

            czw Christopher Waters
            czw Christopher Waters
            Eli Rykoff, Merlin Fisher-Levine
            Christopher Waters, Eli Rykoff, Merlin Fisher-Levine, Patrick Ingraham (Inactive), Robert Lupton, Yousuke Utsumi
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Jenkins

                No builds found.