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

Modernize pybind11 wrapper for afw.detection

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Invalid
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None

      Description

      This is a follow-up ticket to DM-20286, which introduced some code duplication into afw/table/python.h in order to wrap tables one at a time. The duplication cannot be resolved until afw.detection.PeakCatalog is wrapped.

      Unfortunately, afw.detection is so interrelated with afw.image (and possibly afw.fits) that its wrapper cannot be modernized until the others are (specifically, afw.image.exposureInfo depends on afw.detection.psf, and removing the psf module while afw.image uses old-style wrappers causes import failures).

      Modernize the wrapper for afw.detection, and any afw packages necessary to make that possible, then remove the old-style wrappers from python.h.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment -

            I was wrong; afw.detection can be migrated without needing to touch image or fits. I will do so as part of DM-20286, since it will make the merge much cleaner.

            Show
            krzys Krzysztof Findeisen added a comment - I was wrong; afw.detection can be migrated without needing to touch image or fits . I will do so as part of DM-20286 , since it will make the merge much cleaner.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              krzys Krzysztof Findeisen
              Watchers:
              Jim Bosch, Krzysztof Findeisen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.