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

Wrap meas_deblender with pybind11

    Details

      Attachments

        Issue Links

          Activity

          Hide
          fred3m Fred Moolekamp added a comment -

          Pim, do you mind reviewing this ticket for me?

          Show
          fred3m Fred Moolekamp added a comment - Pim, do you mind reviewing this ticket for me?
          Hide
          fred3m Fred Moolekamp added a comment -

          In case we're keeping track of tests needed upstream: there should be a test for the growFootprint method in afw::detection that makes use of the default argument. Apparently meas_deblender is the first place it was used in the stack.

          Show
          fred3m Fred Moolekamp added a comment - In case we're keeping track of tests needed upstream: there should be a test for the growFootprint method in afw::detection that makes use of the default argument. Apparently meas_deblender is the first place it was used in the stack.
          Hide
          pschella Pim Schellart [X] (Inactive) added a comment -

          Looks good. Minor comments on PR.

          Show
          pschella Pim Schellart [X] (Inactive) added a comment - Looks good. Minor comments on PR.
          Hide
          swinbank John Swinbank added a comment -

          Please go ahead and file tickets for tests which you think should be added to afw (or elsewhere in the stack). It's not in scope for this work to write the tests, but we should at least register that they are needed.

          Show
          swinbank John Swinbank added a comment - Please go ahead and file tickets for tests which you think should be added to afw (or elsewhere in the stack). It's not in scope for this work to write the tests, but we should at least register that they are needed.

            People

            • Assignee:
              fred3m Fred Moolekamp
              Reporter:
              pschella Pim Schellart [X] (Inactive)
              Reviewers:
              Pim Schellart [X] (Inactive)
              Watchers:
              Fred Moolekamp, John Swinbank, Pim Schellart [X] (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel