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

Get PSF shape from a source record

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
    • Story Points:
      4
    • Sprint:
      DRP S21b, DRP S22A
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      There is currently no getter method to obtain PSF shape from a SourceRecord using the slots field for PSF shapes. While `base_SdssShape` has a way of obtaining from the key, this doesn't generalize to Hsm Psf moments and cannot be configured. This ticket implements a `getPsfShape` method similar to `getShape` that will return the PSF moments as an `lsst.afw.geom.Quadrupole` object.

        Attachments

          Issue Links

            Activity

            Hide
            kannawad Arun Kannawadi added a comment -

            While I can still construct the PSF shape object from `slot_PsfShape_xx`, `slot_PsfShape_yy` and `slot_PsfShape_xy`, it is cleaner to implement a method in afw that can handle the construction of the `Quadrupole` object.

            Show
            kannawad Arun Kannawadi added a comment - While I can still construct the PSF shape object from `slot_PsfShape_xx`, `slot_PsfShape_yy` and `slot_PsfShape_xy`, it is cleaner to implement a method in afw that can handle the construction of the `Quadrupole` object.
            Hide
            kannawad Arun Kannawadi added a comment -

            This issue is closely related to DM-30359, where `slot_PsfShape` has to be treated differently from `slot_Shape`. Having a common interface for source and PSF shapes would prevent similar confusions from happening.

            Show
            kannawad Arun Kannawadi added a comment - This issue is closely related to DM-30359 , where `slot_PsfShape` has to be treated differently from `slot_Shape`. Having a common interface for source and PSF shapes would prevent similar confusions from happening.
            Hide
            kannawad Arun Kannawadi added a comment -

            Pair-coded this with Jim Bosch. We implemented this as pure-Python methods instead of mimicking the `getShape` methods implemented in C++. Jim says this can be an example for how to extend new methods on afw tables in Python.

            Show
            kannawad Arun Kannawadi added a comment - Pair-coded this with Jim Bosch . We implemented this as pure-Python methods instead of mimicking the `getShape` methods implemented in C++. Jim says this can be an example for how to extend new methods on afw tables in Python.
            Show
            kannawad Arun Kannawadi added a comment - Successful Jenkins run:  https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/35565/pipeline
            Hide
            kannawad Arun Kannawadi added a comment -

            Successful Jenkins run with more minor changes and on ci_imsim and ci_hsc: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/35572/pipeline

            Show
            kannawad Arun Kannawadi added a comment - Successful Jenkins run with more minor changes and on ci_imsim and ci_hsc:  https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/35572/pipeline
            Hide
            kannawad Arun Kannawadi added a comment -

            Thanks for the review Jim. Incorporated the comments and ran Jenkins another time FWIW.

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

            Show
            kannawad Arun Kannawadi added a comment - Thanks for the review Jim. Incorporated the comments and ran Jenkins another time FWIW. https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/35582/pipeline/

              People

              Assignee:
              kannawad Arun Kannawadi
              Reporter:
              kannawad Arun Kannawadi
              Reviewers:
              Jim Bosch
              Watchers:
              Arun Kannawadi, Jim Bosch
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.