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

HSC backport: recent Footprint fixes

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None
    • Story Points:
      2
    • Sprint:
      Science Pipelines DM-S15-3, Science Pipelines DM-S15-4
    • Team:
      Data Release Production

      Description

      This is a backport issue to capture subsequent HSC-side work on features already backported to afw. It includes (so far) the following HSC issues:

        Attachments

          Activity

          Hide
          lauren Lauren MacArthur added a comment -

          I am adding HSC-541 to this issue.

          Show
          lauren Lauren MacArthur added a comment - I am adding HSC-541 to this issue.
          Hide
          lauren Lauren MacArthur added a comment -

          As noted in DM-1945, the fixes here have been exercised by running through all the multiband.py tasks (which do require these fixes to run sucessfully).

          Note that the following from HSC-1215 was imported as part of DM-1945:
          pipe_tasks:

          $ git --no-pager log --stat --reverse NAOJ/master..tickets/HSC-1215
          commit a6271303ce6c85220c0e554cd440343fb65bb3c2
          Author: Jim Bosch <jbosch@astro.princeton.edu>
          Date:   Thu Apr 16 12:28:30 2015 -0400
           
              Sort post-merge Peaks by value before deblending/measuring them.
           
           python/lsst/pipe/tasks/multiBand.py |    6 ++++++
           1 file changed, 6 insertions(+)
          

          And a final commit was added to update the merge flag names to use underscores.

          The changesets can be found on the u/lauren/DM-2606 branch in afw. A simultaneous u/lauren/DM-1945 + u/lauren/DM-2606 Buildbot was successful (although I do believe they would build independently).

          Show
          lauren Lauren MacArthur added a comment - As noted in DM-1945 , the fixes here have been exercised by running through all the multiband.py tasks (which do require these fixes to run sucessfully). Note that the following from HSC-1215 was imported as part of DM-1945 : pipe_tasks: $ git --no-pager log --stat --reverse NAOJ/master..tickets/HSC-1215 commit a6271303ce6c85220c0e554cd440343fb65bb3c2 Author: Jim Bosch <jbosch@astro.princeton.edu> Date: Thu Apr 16 12:28:30 2015 -0400 Sort post-merge Peaks by value before deblending/measuring them. python/lsst/pipe/tasks/multiBand.py | 6 ++++++ 1 file changed, 6 insertions(+) And a final commit was added to update the merge flag names to use underscores. The changesets can be found on the u/lauren/ DM-2606 branch in afw. A simultaneous u/lauren/ DM-1945 + u/lauren/ DM-2606 Buildbot was successful (although I do believe they would build independently).
          Hide
          jbosch Jim Bosch added a comment -

          Everything here looks good. You're welcome to merge this now if you're very confident it won't break anything without DM-1945, or you can wait until I'm done with that one.

          Show
          jbosch Jim Bosch added a comment - Everything here looks good. You're welcome to merge this now if you're very confident it won't break anything without DM-1945 , or you can wait until I'm done with that one.
          Hide
          lauren Lauren MacArthur added a comment -

          Thanks. I'll fire off a Buildbot just to be sure!

          Show
          lauren Lauren MacArthur added a comment - Thanks. I'll fire off a Buildbot just to be sure!
          Hide
          price Paul Price added a comment -

          Do you want to add Footprint features about to be added via HSC-1274?

          Show
          price Paul Price added a comment - Do you want to add Footprint features about to be added via HSC-1274?
          Hide
          lauren Lauren MacArthur added a comment -

          I imagine we will...and it will be part of a future ticket as we move on with our inventory.

          Show
          lauren Lauren MacArthur added a comment - I imagine we will...and it will be part of a future ticket as we move on with our inventory.
          Hide
          swinbank John Swinbank added a comment -

          Lauren MacArthur – when you get chance, could you add a few words describing what's been done here to the release notes at https://confluence.lsstcorp.org/display/DM/Data+Release+Production+WIP+S15+release+notes? Thanks!

          Show
          swinbank John Swinbank added a comment - Lauren MacArthur – when you get chance, could you add a few words describing what's been done here to the release notes at https://confluence.lsstcorp.org/display/DM/Data+Release+Production+WIP+S15+release+notes? Thanks!

            People

            • Assignee:
              lauren Lauren MacArthur
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Jim Bosch
              Watchers:
              Jim Bosch, John Swinbank, Lauren MacArthur, Paul Price
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel