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

Wrap obs_base (was daf_butlerUtils) with pybind11

    Details

      Description

      May not require any wrapping, if so then this ticket is just for tracking. Otherwise, adjust SP.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment - - edited

            Has a non-trivial dependency on `cameraGeom`, and possibly other code.

            Show
            krzys Krzysztof Findeisen added a comment - - edited Has a non-trivial dependency on `cameraGeom`, and possibly other code.
            Hide
            krzys Krzysztof Findeisen added a comment -

            coadd_utils depends on obs_base, so obs_base should be done first.

            Show
            krzys Krzysztof Findeisen added a comment - coadd_utils depends on obs_base , so obs_base should be done first.
            Hide
            krzys Krzysztof Findeisen added a comment -

            obs_test and pipe_base depend on obs_base, so obs_base should be done first

            Show
            krzys Krzysztof Findeisen added a comment - obs_test and pipe_base depend on obs_base , so obs_base should be done first
            Hide
            krzys Krzysztof Findeisen added a comment -

            Hi Pim Schellart [X], can you do a quick review of this ticket? The changes themselves are small, but they deviate a bit from the "normal" wrapping workflow.

            Show
            krzys Krzysztof Findeisen added a comment - Hi Pim Schellart [X] , can you do a quick review of this ticket? The changes themselves are small, but they deviate a bit from the "normal" wrapping workflow.
            Hide
            pschella Pim Schellart [X] (Inactive) added a comment -

            Looks good. One minor comment on PR.

            Show
            pschella Pim Schellart [X] (Inactive) added a comment - Looks good. One minor comment on PR.
            Hide
            krzys Krzysztof Findeisen added a comment -

            Merged after removing unnecessary scaffolding.

            Show
            krzys Krzysztof Findeisen added a comment - Merged after removing unnecessary scaffolding.

              People

              • Assignee:
                krzys Krzysztof Findeisen
                Reporter:
                pschella Pim Schellart [X] (Inactive)
                Reviewers:
                Pim Schellart [X] (Inactive)
                Watchers:
                Krzysztof Findeisen, Pim Schellart [X] (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel