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

Update Gen3 isrTask for BF corrections

    Details

      Description

      isrTask's runQuantum() method should have been updated in DM-22659 but I didn't know how to do it, and Christopher Waters kindly said he'd do it if I ticketed and {{TODO:}}ed it.

        Attachments

          Issue Links

            Activity

            Hide
            dtaranu Dan Taranu added a comment - - edited
            Show
            dtaranu Dan Taranu added a comment - - edited Jenkins run: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/30967/pipeline Merlin Fisher-Levine , can you send an example using new-style BF kernels to test this?
            Hide
            mfisherlevine Merlin Fisher-Levine added a comment -

            You just want a link to a rerun that contains a new style kernel? If so, then yes:

            /datasets/hsc/repo/rerun/private/mfl/DM-18683_allDets

            Not claiming they kernels are great, but they should be compliant. If you wanted an example of actually running ISR in Gen3 then I've not yet done that, so don't have an easy command to paste I'm afraid... Will that do for now?

            Show
            mfisherlevine Merlin Fisher-Levine added a comment - You just want a link to a rerun that contains a new style kernel? If so, then yes: /datasets/hsc/repo/rerun/private/mfl/ DM-18683 _allDets Not claiming they kernels are great, but they should be compliant. If you wanted an example of actually running ISR in Gen3 then I've not yet done that, so don't have an easy command to paste I'm afraid... Will that do for now?
            Hide
            mfisherlevine Merlin Fisher-Levine added a comment -

            Ignore the strike-through �

            Show
            mfisherlevine Merlin Fisher-Levine added a comment - Ignore the strike-through �
            Hide
            nlust Nate Lust added a comment -

            A few comments on github

            Show
            nlust Nate Lust added a comment - A few comments on github

              People

              • Assignee:
                czw Christopher Waters
                Reporter:
                mfisherlevine Merlin Fisher-Levine
                Reviewers:
                Nate Lust
                Watchers:
                Christopher Waters, Dan Taranu, Merlin Fisher-Levine, Nate Lust
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: