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

Add macro for Pybind11 wrapping of LSST_CONTROL_FIELD

    XMLWordPrintable

    Details

      Description

      The LSST_CONTROL_FIELD macro defines multiple fields and methods in a Control object's C++ interface, most of which are implementation details hidden from the average programmer. These implementation methods must appear in an object's Python interface for pex_config to work correctly, but should not be wrapped manually as this would break the abstraction provided by the macro.

      The simplest solution is to define a macro that adds all elements of an LSST_CONTROL_FIELD to the appropriate Pybind11 wrapper object. This macro must be defined in pex_config in a place where it can be included by the wrapper code for any other stack package.

        Attachments

          Issue Links

          blocks

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-7797 Wrap afw::image with pybind11

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-8264 Finish wrapping afw::table with pybind11

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-8447 Wrap afw::image with pybind11 (UW work)

          • Done

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            krzys Krzysztof Findeisen created issue -
            krzys Krzysztof Findeisen made changes -
            Field Original Value New Value
            Epic Link DM-7362 [ 26448 ]
            krzys Krzysztof Findeisen made changes -
            Link This issue blocks DM-7797 [ DM-7797 ]
            krzys Krzysztof Findeisen made changes -
            Link This issue blocks DM-8264 [ DM-8264 ]
            krzys Krzysztof Findeisen made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Please review the newly standardized and centralized macro. Thanks!

            Show
            krzys Krzysztof Findeisen added a comment - Please review the newly standardized and centralized macro. Thanks!
            krzys Krzysztof Findeisen made changes -
            Reviewers Pim Schellart [ pschella ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            krzys Krzysztof Findeisen made changes -
            Link This issue is blocked by DM-8374 [ DM-8374 ]
            krzys Krzysztof Findeisen made changes -
            Labels Pybind11 SciencePipelines
            krzys Krzysztof Findeisen made changes -
            Link This issue is blocked by DM-8374 [ DM-8374 ]
            Hide
            Permalink
            pschella Pim Schellart [X] (Inactive) added a comment -

            Looks fine to me. Just be sure to merge it into the ticket branch instead of master.

            Show
            pschella Pim Schellart [X] (Inactive) added a comment - Looks fine to me. Just be sure to merge it into the ticket branch instead of master.
            pschella Pim Schellart [X] (Inactive) made changes -
            Resolution Done [ 10000 ]
            Status In Review [ 10004 ] Done [ 10002 ]
            Hide
            Permalink
            pschella Pim Schellart [X] (Inactive) added a comment -

            Something was wrong with JIRA on my machine. Should be Review Complete, not Done.

            Show
            pschella Pim Schellart [X] (Inactive) added a comment - Something was wrong with JIRA on my machine. Should be Review Complete, not Done.
            pschella Pim Schellart [X] (Inactive) made changes -
            Resolution Done [ 10000 ]
            Status Done [ 10002 ] Reviewed [ 10101 ]
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Is tickets/DM-6168 the wrong target?

            Show
            krzys Krzysztof Findeisen added a comment - Is tickets/ DM-6168 the wrong target?
            krzys Krzysztof Findeisen made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            krzys Krzysztof Findeisen made changes -
            Link This issue blocks DM-8447 [ DM-8447 ]

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              krzys Krzysztof Findeisen
              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

                  No builds found.