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

            No builds found.
            pschella Pim Schellart [X] (Inactive) created issue -
            pschella Pim Schellart [X] (Inactive) made changes -
            Field Original Value New Value
            Epic Link DM-7717 [ 26925 ]
            pschella Pim Schellart [X] (Inactive) made changes -
            Link This issue blocks DM-8419 [ DM-8419 ]
            pschella Pim Schellart [X] (Inactive) made changes -
            Summary Wrap daf_butlerUtils with pybind11 Wrap obs_base (was daf_butlerUtils) with pybind11
            krzys Krzysztof Findeisen made changes -
            Assignee Krzysztof Findeisen [ krzys ]
            krzys Krzysztof Findeisen made changes -
            Component/s obs_base [ 10719 ]
            Team Alert Production [ 10300 ]
            Labels Pybind11 SciencePipelines
            krzys Krzysztof Findeisen made changes -
            Epic Link DM-7717 [ 26925 ] DM-8450 [ 28066 ]
            krzys Krzysztof Findeisen made changes -
            Sprint Alert Production S17 - 12 [ 305 ]
            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.
            krzys Krzysztof Findeisen made changes -
            Link This issue is blocked by DM-7801 [ DM-7801 ]
            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.
            krzys Krzysztof Findeisen made changes -
            Link This issue blocks DM-8420 [ DM-8420 ]
            krzys Krzysztof Findeisen made changes -
            Link This issue is blocked by DM-8424 [ DM-8424 ]
            krzys Krzysztof Findeisen made changes -
            Link This issue blocks DM-8422 [ DM-8422 ]
            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
            krzys Krzysztof Findeisen made changes -
            Link This issue blocks DM-8421 [ DM-8421 ]
            krzys Krzysztof Findeisen made changes -
            Story Points 0.1 4
            krzys Krzysztof Findeisen made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            krzys Krzysztof Findeisen made changes -
            Link This issue is blocked by DM-8654 [ DM-8654 ]
            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.
            krzys Krzysztof Findeisen made changes -
            Reviewers Pim Schellart [ pschella ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            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.
            pschella Pim Schellart [X] (Inactive) made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            Hide
            krzys Krzysztof Findeisen added a comment -

            Merged after removing unnecessary scaffolding.

            Show
            krzys Krzysztof Findeisen added a comment - Merged after removing unnecessary scaffolding.
            krzys Krzysztof Findeisen made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              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:

                  Jenkins Builds

                  No builds found.