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

Fix dimension ordering in decorrelation afterburner

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ip_diffim
    • Labels:
      None
    • Story Points:
      1
    • Epic Link:
    • Sprint:
      AP S22-6 (May)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      Testing DM-33001, I came across an exposure that had a decorrelated PSF with shape (41,43) pixels. The decorrelation afterburner assumed that the ordering of x vs y in numpy arrays and lsst.geom.Extent2I are the same, but this is a bug.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          No builds found.
          sullivan Ian Sullivan created issue -
          sullivan Ian Sullivan made changes -
          Field Original Value New Value
          Status To Do [ 10001 ] In Progress [ 3 ]
          Hide
          Permalink
          sullivan Ian Sullivan added a comment -

          Pull request: https://github.com/lsst/ip_diffim/pull/219

          Show
          sullivan Ian Sullivan added a comment - Pull request: https://github.com/lsst/ip_diffim/pull/219
          Hide
          Permalink
          sullivan Ian Sullivan added a comment -

          Could you please do a quick review of this one line bug fix?

          Show
          sullivan Ian Sullivan added a comment - Could you please do a quick review of this one line bug fix?
          sullivan Ian Sullivan made changes -
          Reviewers Kenneth Herner [ kherner ]
          Status In Progress [ 3 ] In Review [ 10004 ]
          Hide
          Permalink
          sullivan Ian Sullivan added a comment -

          Reviewed on Github.

          Show
          sullivan Ian Sullivan added a comment - Reviewed on Github.
          sullivan Ian Sullivan made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          sullivan Ian Sullivan made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]

            People

            Assignee:
            sullivan Ian Sullivan
            Reporter:
            sullivan Ian Sullivan
            Reviewers:
            Kenneth Herner
            Watchers:
            Ian Sullivan, Kenneth Herner
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.