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

Use CoaddPsfs from all warps in getTemplate task

    XMLWordPrintable

    Details

    • Story Points:
      12
    • Sprint:
      AP S21-6 (May), AP F21-1 (June), AP F21-2 (July), AP F21-3 (August)
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      Currently GetCoaddAsTemplateTask sets the PSF of the template with that of the first warp that it adds in. Bob Armstrong improved this on https://github.com/LSSTDESC/dia_pipe/blob/master/python/lsst/dia/pipe/getMultiTractTemplate.py. However, he reports an edge case where it fails trying to compute the average position of the composite PSF near edges because the average position can be in a region of no data.

      Bob Armstrong will ask Bruno for tract/patch that failed as a how-to-reproduce.

      This ticket covers ensuring that the PSF is valid across the whole template

        Attachments

          Issue Links

            Activity

            yusra Yusra AlSayyad created issue -
            yusra Yusra AlSayyad made changes -
            Field Original Value New Value
            Epic Link PREOPS-450 [ 463950 ]
            sullivan Ian Sullivan made changes -
            Assignee John Parejko [ parejkoj ]
            sullivan Ian Sullivan made changes -
            Sprint AP S21-5 (April) [ 1084 ]
            Team Alert Production [ 10300 ]
            Parejkoj John Parejko made changes -
            Description Currently getTemplate task sets the PSF of the template with that of the first warp that it adds in. [~rearmstr] improved this on https://github.com/LSSTDESC/dia_pipe/blob/master/python/lsst/dia/pipe/getMultiTractTemplate.py. However, he reports an edge case where it fails trying to compute the average position of the composite PSF near edges because the average position can be in a region of no data.

            [~rearmstr] will ask Bruno for tract/patch that failed as a how-to-reproduce.

            This ticket covers ensuring that the PSF is valid across the whole template
            Currently {{GetCoaddAsTemplateTask}} sets the PSF of the template with that of the first warp that it adds in. [~rearmstr] improved this on https://github.com/LSSTDESC/dia_pipe/blob/master/python/lsst/dia/pipe/getMultiTractTemplate.py. However, he reports an edge case where it fails trying to compute the average position of the composite PSF near edges because the average position can be in a region of no data.

            [~rearmstr] will ask Bruno for tract/patch that failed as a how-to-reproduce.

            This ticket covers ensuring that the PSF is valid across the whole template
            yusra Yusra AlSayyad made changes -
            Remote Link This issue links to "Page (Confluence)" [ 28284 ]
            sullivan Ian Sullivan made changes -
            Story Points 6
            sullivan Ian Sullivan made changes -
            Sprint AP S21-5 (April) [ 1084 ] AP S21-6 (May) [ 1088 ]
            sullivan Ian Sullivan made changes -
            Rank Ranked higher
            yusra Yusra AlSayyad made changes -
            yusra Yusra AlSayyad made changes -
            Assignee John Parejko [ parejkoj ] Yusra AlSayyad [ yusra ]
            yusra Yusra AlSayyad made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            sullivan Ian Sullivan made changes -
            Link This issue relates to DM-30496 [ DM-30496 ]
            sullivan Ian Sullivan made changes -
            Sprint AP S21-6 (May) [ 1088 ] AP S21-6 (May), AP F21-1 (June) [ 1088, 1096 ]
            yusra Yusra AlSayyad made changes -
            Epic Link PREOPS-450 [ 463950 ] PREOPS-477 [ 476909 ]
            sullivan Ian Sullivan made changes -
            Sprint AP S21-6 (May), AP F21-1 (June) [ 1088, 1096 ] AP S21-6 (May), AP F21-1 (June), AP F21-2 (July) [ 1088, 1096, 1102 ]
            yusra Yusra AlSayyad made changes -
            Attachment sourceDensity29310.png [ 51131 ]
            yusra Yusra AlSayyad made changes -
            Attachment sourceDensityDM-29310.png [ 51132 ]
            yusra Yusra AlSayyad made changes -
            Attachment sourceDensity29310.png [ 51131 ]
            yusra Yusra AlSayyad made changes -
            Attachment sourceDensityw_2021_28.png [ 51133 ]
            yusra Yusra AlSayyad made changes -
            Reviewers John Parejko [ parejkoj ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            yusra Yusra AlSayyad made changes -
            Reviewers John Parejko [ parejkoj ] Ian Sullivan [ sullivan ]
            sullivan Ian Sullivan made changes -
            Sprint AP S21-6 (May), AP F21-1 (June), AP F21-2 (July) [ 1088, 1096, 1102 ] AP S21-6 (May), AP F21-1 (June), AP F21-2 (July), AP F21-3 (August) [ 1088, 1096, 1102, 1109 ]
            sullivan Ian Sullivan made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            yusra Yusra AlSayyad made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            yusra Yusra AlSayyad made changes -
            Story Points 6 12
            yusra Yusra AlSayyad made changes -
            Team Alert Production [ 10300 ] Data Release Production [ 10301 ]

              People

              Assignee:
              yusra Yusra AlSayyad
              Reporter:
              yusra Yusra AlSayyad
              Reviewers:
              Ian Sullivan
              Watchers:
              Bob Armstrong, Ian Sullivan, John Parejko, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  CI Builds

                  No builds found.