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

Implement SpanSet core functionality

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • afw
    • None
    • 6
    • DRP F16-4, DRP F16-5, DRP F16-6
    • 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

            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.

            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.
            nlust Nate Lust added a comment -

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

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

            merged to master

            nlust Nate Lust added a comment - merged to master

            People

              nlust Nate Lust
              jbosch Jim Bosch
              Bob Armstrong
              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:

                Jenkins

                  No builds found.