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

Fix minor bug in Gen2 DCR diffim template generation

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ip_diffim
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      AP F20-4 (September)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      sensorRef.datasetExists(**patchArgDict) fails in getTemplate of ip_diffim for DCR coadds, because it needs to have a subfilter defined in the patchArgDict. Including subfilter isn't a problem for non-DCR coadds. This only affects Gen2.

        Attachments

          Activity

          Hide
          sullivan Ian Sullivan added a comment -

          This is a single line addition that should be trivial, but I would appreciate you taking a quick look before I merge.

          PR here: https://github.com/lsst/ip_diffim/pull/165

          Currently running Jenkins here: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/32568/pipeline

          Show
          sullivan Ian Sullivan added a comment - This is a single line addition that should be trivial, but I would appreciate you taking a quick look before I merge. PR here: https://github.com/lsst/ip_diffim/pull/165 Currently running Jenkins here:  https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/32568/pipeline
          Hide
          gkovacs Gabor Kovacs added a comment -

          Looks good. Please see github for some line specific comments.

          Show
          gkovacs Gabor Kovacs added a comment - Looks good. Please see github for some line specific comments.

            People

            • Assignee:
              sullivan Ian Sullivan
              Reporter:
              sullivan Ian Sullivan
              Reviewers:
              Gabor Kovacs
              Watchers:
              Gabor Kovacs, Ian Sullivan
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: