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

Bug in coaddDriver when selecting images by PSF quality.

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_drivers
    • Labels:

      Description

      In DM-9855, the option to select input images based on PSF quality was added and made the default for assembleCoadd in obs_subaru. However, coaddDriver has its own function to select images and sets the one in assemble coadd to null. The config in obs_subaru was overriding this and the selection of images was being run twice and having strange results.

      The resolution is to set the image selector in coaddDriver to the PSF quality selector and set the assembleCoadd selector to null.

        Attachments

          Issue Links

          blocks

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-10129 Process HSC RC dataset using Stack version chosen for the full S17B reprocessing

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-10266 Release hscPipe 5.0-beta4

          • Done
          is triggered by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-9855 Select images for coadd based on PSF quality.

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            jbosch Jim Bosch added a comment -

            I've rebased this on master as u/jbosch/DM-10235 (no conflicts), and the changes look good to me. Please go ahead and merge if you think it's done.

            Show
            jbosch Jim Bosch added a comment - I've rebased this on master as u/jbosch/ DM-10235 (no conflicts), and the changes look good to me. Please go ahead and merge if you think it's done.
            Hide
            Permalink
            rearmstr Bob Armstrong added a comment -

            Merged.

            Show
            rearmstr Bob Armstrong added a comment - Merged.

              People

              Assignee:
              rearmstr Bob Armstrong
              Reporter:
              rearmstr Bob Armstrong
              Watchers:
              Bob Armstrong, Jim Bosch
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.