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

Move Record printing to C++

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None
    • Story Points:
      0.5
    • Sprint:
      DRP S18-4
    • Team:
      Data Release Production

      Description

      Current implementation is apparently super slow, and looking at what it's doing (constructing a new dictionary of all fields via extract just to get one element from it, for every field), I can see why.

      That could be cleaned up in Python but a lot of it is more naturally and definitely more efficient in C++, and doing that will give us a C++ operator<<, too, which is nice.

        Attachments

          Activity

          Hide
          jbosch Jim Bosch added a comment -

          Nate Lust, mind looking at this small move of code from Python to C++?

           

          Show
          jbosch Jim Bosch added a comment - Nate Lust , mind looking at this small move of code from Python to C++?  
          Hide
          nlust Nate Lust added a comment -

          comment up on github, curious about your take, will mark as reviewed, but please respond to the comment before merge.

          Show
          nlust Nate Lust added a comment - comment up on github, curious about your take, will mark as reviewed, but please respond to the comment before merge.
          Hide
          jbosch Jim Bosch added a comment -

          Merged after implementing Nate Lust's suggestion.

          Show
          jbosch Jim Bosch added a comment - Merged after implementing Nate Lust 's suggestion.

            People

            • Assignee:
              jbosch Jim Bosch
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Nate Lust
              Watchers:
              Jim Bosch, Nate Lust
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel