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

Provide utility function for wrapping operator<<

    Details

    • Story Points:
      3
    • Sprint:
      AP S18-2, AP S18-3
    • Team:
      Alert Production

      Description

      We frequently want to implement a class's __str__ or __repr__ methods in terms of its operator<<. The current approach has led to a fair amount of code duplication throughout the stack.

      Once a function template that takes a printable object and returns a string is available, explicit wrappers for operator<< should be rewritten to use it.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment - - edited

            Hi Pim Schellart [X], can you review this refactoring of some of the pybind11 code? It's about 100 lines across 4 packages. Thanks!

            Edit: PR is lsst/utils#47 and references therein.

            Show
            krzys Krzysztof Findeisen added a comment - - edited Hi Pim Schellart [X] , can you review this refactoring of some of the pybind11 code? It's about 100 lines across 4 packages. Thanks! Edit: PR is lsst/utils#47 and references therein.
            Hide
            pschella Pim Schellart [X] (Inactive) added a comment -

            Minor naming convention issue, otherwise ok.

            Show
            pschella Pim Schellart [X] (Inactive) added a comment - Minor naming convention issue, otherwise ok.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel