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

Implement SpanSet core functionality

    XMLWordPrintable

    Details

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

      Description

      Implement the core of the SpanSet class, as prototyped in RFC-37. This includes the following:

      • All STL container methods and typedefs, and their Pythonic counterparts.
      • All constructors and assignment operators, except for SpanSetBuilder. This includes the ability to detect and fix overlapping Spans.
      • All simple accessors.
      • isContiguous()
      • The shift and clip methods.

        Attachments

          Issue Links

            Activity

            Hide
            nlust Nate Lust added a comment -

            There is no single ticket that is best to review this work as it is spread over many tickets, so this will serve well enough. Thank you for agreeing to do this review. The work is all on the tickets/DM-7170 branch on afw. For your part of the review, please review SpanSets.h, SpanSets.cc, and the unit test. Within SpanSets.h you are not responsible for reviewing the applyFunctor(Impl) functions, just the normal class method. There are a few class methods which make use of that functionality, and you can decide for yourself if you want to look into those or not. If there are any questions, please send me a message here or on Slack.

            Show
            nlust Nate Lust added a comment - There is no single ticket that is best to review this work as it is spread over many tickets, so this will serve well enough. Thank you for agreeing to do this review. The work is all on the tickets/ DM-7170 branch on afw. For your part of the review, please review SpanSets.h, SpanSets.cc, and the unit test. Within SpanSets.h you are not responsible for reviewing the applyFunctor(Impl) functions, just the normal class method. There are a few class methods which make use of that functionality, and you can decide for yourself if you want to look into those or not. If there are any questions, please send me a message here or on Slack.
            Hide
            nlust Nate Lust added a comment -

            Bob, I have made updates to this code. Would you like to look back over it?

            Show
            nlust Nate Lust added a comment - Bob, I have made updates to this code. Would you like to look back over it?
            Hide
            nlust Nate Lust added a comment -

            merged to master

            Show
            nlust Nate Lust added a comment - merged to master

              People

              Assignee:
              nlust Nate Lust
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Bob Armstrong
              Watchers:
              Bob Armstrong, Jim Bosch, Nate Lust, Pim Schellart [X] (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  CI Builds

                  No builds found.