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

Enable running coadds on arbitrary list of tempExps

    Details

    • Story Points:
      1
    • Sprint:
      DRP S18-3, DRP S18-4, DRP S18-5, DRP S18-6, DRP F18-1, DRP F18-2, DRP F18-3, DRP F18-4, DRP F18-5, DRP F18-6, DRP S19-1
    • Team:
      Data Release Production

      Description

      While debugging coaddition, I have an alternate API for AssembleCoadd.run() that I use in my local copy of pipe_tasks so that I can quickly coadd any list of tempExp DataRefs in a notebook without going through the long "selectExposures" task which converts a list of calexp data refs to a list of temp exp dataRefs. Would this be useful to anyone else?

           @pipeBase.timeMethod
      -    def run(self, dataRef, selectDataList=[]):
      +    def run(self, dataRef, selectDataList=[], tempExpRefList=None):
               """!
               \brief Assemble a coadd from a set of Warps
       
      @@ -327,13 +327,14 @@ discussed in \ref pipeTasks_multiBand (but note that normally, one would use the
                        - nImage: exposure count image
               """
               skyInfo = self.getSkyInfo(dataRef)
      -        calExpRefList = self.selectExposures(dataRef, skyInfo, selectDataList=selectDataList)
      -        if len(calExpRefList) == 0:
      -            self.log.warn("No exposures to coadd")
      -            return
      -        self.log.info("Coadding %d exposures", len(calExpRefList))
      -
      -        tempExpRefList = self.getTempExpRefList(dataRef, calExpRefList)
      +        if tempExpRefList is None:
      +            calExpRefList = self.selectExposures(dataRef, skyInfo, selectDataList=selectDataList)
      +            if len(calExpRefList) == 0:
      +                self.log.warn("No exposures to coadd")
      +                return
      +            self.log.info("Coadding %d exposures", len(calExpRefList))
      +
      +            tempExpRefList = self.getTempExpRefList(dataRef, calExpRefList)
      

        Attachments

          Issue Links

            Activity

            Hide
            sullivan Ian Sullivan added a comment -

            I've started using these changes frequently for testing in DM-9615, so if you complete this ticket and merge the changes it would help me.

            Show
            sullivan Ian Sullivan added a comment - I've started using these changes frequently for testing in DM-9615 , so if you complete this ticket and merge the changes it would help me.
            Hide
            yusra Yusra AlSayyad added a comment -

            This change will appear on DM-17045.

            Show
            yusra Yusra AlSayyad added a comment - This change will appear on DM-17045 .

              People

              • Assignee:
                yusra Yusra AlSayyad
                Reporter:
                yusra Yusra AlSayyad
                Watchers:
                Ian Sullivan, Yusra AlSayyad
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel