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

Fix minor bug in Gen2 DCR diffim template generation

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ip_diffim
    • Labels:
      None
    • Story Points:
      1
    • Epic Link:
    • 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

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          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
          Permalink
          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: