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

Update forcedPhotCoadd to use the correct input image in Gen3

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Epic Link:
    • Sprint:
      DRP S22A
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      The forcedPhotCoadd function currently has the input coadd image connection specified as

      name="{inputCoaddName}Coadd"

      this should be

      name="{inputCoaddName}Coadd_calexp"

      For reference, the descriptions are:

      deepCoadd:
      		 
          description: "A coadd assembled from direct warps, produced by AssembleCoaddTask."


      deepCoadd_calexp:
      		 
          description: >
      		 
            An exposure that holds stacked data, after DetectCoaddSourcesTask
      		 
            (thus the background has been subtracted).

      Given the importance of this update, backport to v23 is being requested for the ongoing DP0.2 processing, so would need merging before moving on to the coaddition stages.

        Attachments

          Issue Links

          mentioned in

          Page Loading...

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            lauren Lauren MacArthur added a comment -

            Jenkins is running.

            Show
            lauren Lauren MacArthur added a comment - Jenkins is running .
            Hide
            Permalink
            lauren Lauren MacArthur added a comment -

            This commit was approved on DM-30284, so I went ahead and merged.  Since this will be on your radar for backporting anyway, I figured I’d ask you to officially review the ticket.

            Show
            lauren Lauren MacArthur added a comment - This commit was approved on DM-30284 , so I went ahead and merged.  Since this will be on your radar for backporting anyway, I figured I’d ask you to officially review the ticket.
            Hide
            Permalink
            lauren Lauren MacArthur added a comment - - edited

            I have created the tickets/DM-33195-v23 branch according to the backporting instructions and have a Jenkins with:

            PRODUCTS: lsst_distrib lsst_ci ci_imsim ci_hsc
            		 
            REFS: tickets/DM-33195-v23 v23.0.x v23.0.0.rc1

            running.

            Show
            lauren Lauren MacArthur added a comment - - edited I have created the tickets/ DM-33195 -v23 branch according to the backporting instructions and have a Jenkins with: PRODUCTS: lsst_distrib lsst_ci ci_imsim ci_hsc REFS: tickets / DM - 33195 - v23 v23. 0.x v23. 0.0 .rc1 running .
            Hide
            Permalink
            lauren Lauren MacArthur added a comment -

            Clean Jenkins. Am I good to merge the backport Yusra AlSayyad (I asked for a review of the PR as I’m feeling trigger shy…)?

            Show
            lauren Lauren MacArthur added a comment - Clean  Jenkins . Am I good to merge the backport Yusra AlSayyad (I asked for a review of the PR as I’m feeling trigger shy…)?
            Hide
            Permalink
            lauren Lauren MacArthur added a comment -

            Thanks for the sanity check!  Backport merged to v23.0.x

            Show
            lauren Lauren MacArthur added a comment - Thanks for the sanity check!  Backport merged to v23.0.x

              People

              Assignee:
              lauren Lauren MacArthur
              Reporter:
              lauren Lauren MacArthur
              Reviewers:
              Yusra AlSayyad
              Watchers:
              Colin Slater, Jim Bosch, Lauren MacArthur, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.