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

Audit pybind11 port for outstanding technical debt

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Story Points:
      3
    • Sprint:
      DRP S17-2, DRP S17-3
    • Team:
      Data Release Production

      Description

      During the pybind11 porting work, temporary solutions have been adopted in some places to enable us to make rapid progress. Particularly in parts of the codebase which we expect to be performance critical, these need to be cleaned up and, if necessary, replaced with something better. Jim Bosch has volunteered to audit the work, identify likely problem areas, and file tickets for clean-up.

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment - - edited

            This issue has spawned a lot of tickets (including many not related to pybind11), and I'm not planning to do any more reviewing of the pybind11 branch as it stands now - for packages outside afw, I imagine the issues that need fixing will be the same, and it'll be easier to fix them at the same time we look for them. DM-9182 contains the cleanup checklist that represents the primary outcome of this audit.

            Show
            jbosch Jim Bosch added a comment - - edited This issue has spawned a lot of tickets (including many not related to pybind11), and I'm not planning to do any more reviewing of the pybind11 branch as it stands now - for packages outside afw, I imagine the issues that need fixing will be the same, and it'll be easier to fix them at the same time we look for them. DM-9182 contains the cleanup checklist that represents the primary outcome of this audit.

              People

              • Assignee:
                jbosch Jim Bosch
                Reporter:
                swinbank John Swinbank
                Watchers:
                Jim Bosch, John Swinbank, Krzysztof Findeisen
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel