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

Clean up multi-component Footprints

    XMLWordPrintable

    Details

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

      Description

      Following the landing of DM-1545, it's possible for an erosion operation on a footprint to cause it to split into multiple components and for peaks which were previously inside the footprint to not fall inside any of those components.

      Here, we should provide a "clean up" operation that takes a multiple-component footprint and splits it into a set of contiguous footprints with appropriate peak lists.

        Attachments

          Issue Links

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-1545 Span-based shrink operations for Footprint

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            swinbank John Swinbank added a comment -

            Nate Lust, will this still be an issue with the redesigned footprint class?

            Show
            swinbank John Swinbank added a comment - Nate Lust , will this still be an issue with the redesigned footprint class?
            Hide
            Permalink
            nlust Nate Lust added a comment -

            I don't know why I didn't see the comment on this all that time ago, this ticket will still need to be implemented, but will be very quick to do, as SpanSet has a method which does 90% of the work now.

            Show
            nlust Nate Lust added a comment - I don't know why I didn't see the comment on this all that time ago, this ticket will still need to be implemented, but will be very quick to do, as SpanSet has a method which does 90% of the work now.
            Hide
            Permalink
            nlust Nate Lust added a comment -

            Pull request created at https://github.com/lsst/afw/pull/159

            Show
            nlust Nate Lust added a comment - Pull request created at https://github.com/lsst/afw/pull/159
            Hide
            Permalink
            jbosch Jim Bosch added a comment -

            Looks good. Only comments are whitespace quibbles.

            Show
            jbosch Jim Bosch added a comment - Looks good. Only comments are whitespace quibbles.
            Hide
            Permalink
            nlust Nate Lust added a comment -

            merge to epic branch

            Show
            nlust Nate Lust added a comment - merge to epic branch

              People

              Assignee:
              nlust Nate Lust
              Reporter:
              swinbank John Swinbank
              Reviewers:
              Jim Bosch
              Watchers:
              Jim Bosch, John Swinbank, Nate Lust
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.