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

Provide utility function for wrapping operator<<

    XMLWordPrintable

    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:

                  CI Builds

                  No builds found.