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

Fix DCR multiband bugs introduced by new deblender

    XMLWordPrintable

    Details

      Description

      The recent changes to the deblender broke multiband source measurement on DCR coadds. This ticket is to identify and fix the breakages.

        Attachments

          Issue Links

            Activity

            Hide
            sullivan Ian Sullivan added a comment -

            Despite the title of the ticket, only the creation of the two `Deblend...` classes and the additions to `obs_base` were required by the new deblender. The rest were caused by gradual changes to `pipe_tasks` that I hadn't caught until trying to re-run older code.

            Also, please note that `pipe_tasks` does not show up in the pull requests link, but does have a pull request. That package has so many pull requests that JIRA has stopped trying to link them.

            Show
            sullivan Ian Sullivan added a comment - Despite the title of the ticket, only the creation of the two `Deblend...` classes and the additions to `obs_base` were required by the new deblender. The rest were caused by gradual changes to `pipe_tasks` that I hadn't caught until trying to re-run older code. Also, please note that `pipe_tasks` does not show up in the pull requests link, but does have a pull request. That package has so many pull requests that JIRA has stopped trying to link them.
            Hide
            czw Christopher Waters added a comment -

            My only concern was that the `obs_base` changes look to have a typo that defined `deepCoadd_deblendedModel` twice.  Everything else looks good.

            Show
            czw Christopher Waters added a comment - My only concern was that the `obs_base` changes look to have a typo that defined `deepCoadd_deblendedModel` twice.  Everything else looks good.

              People

              Assignee:
              sullivan Ian Sullivan
              Reporter:
              sullivan Ian Sullivan
              Reviewers:
              Christopher Waters
              Watchers:
              Christopher Waters, Ian Sullivan, John Swinbank
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: