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

Wrap obs_base (was daf_butlerUtils) with pybind11

    XMLWordPrintable

    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:

                  CI Builds

                  No builds found.