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

Write brief guide to building PSF-matched coadds

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      DRP S17-5, DRP S17-6
    • Team:
      Data Release Production

      Description

      Please provide a ~page of hints for setting appropriate parameters when generating PSF matched coadds.

      Per discussion of 2017-03-29, we reckon this should go into Doxygen.

        Attachments

          Activity

          Hide
          swinbank John Swinbank added a comment -

          Should include documenting the need to set:

          config.warpAndPsfMatch.psfMatch.kernel['AL'].scaleByFwhm=False
          

          (And, if you like, adding a setDefaults block to do that for you).

          Show
          swinbank John Swinbank added a comment - Should include documenting the need to set: config.warpAndPsfMatch.psfMatch.kernel['AL'].scaleByFwhm=False (And, if you like, adding a setDefaults block to do that for you).
          Hide
          yusra Yusra AlSayyad added a comment -

          David, if you have some time would you have a look? The pull requests are a delta on DM-8491, which is waiting for a final green light to merge. I will wait to merge this ticket until that one is in.

          I didn't get carried away writing MakeCoaddTempExpTask Task Documentation, but rather put the skeleton in as a place to put the hints for PSF-matched coadd configs.

          Show
          yusra Yusra AlSayyad added a comment - David, if you have some time would you have a look? The pull requests are a delta on DM-8491 , which is waiting for a final green light to merge. I will wait to merge this ticket until that one is in. I didn't get carried away writing MakeCoaddTempExpTask Task Documentation, but rather put the skeleton in as a place to put the hints for PSF-matched coadd configs.
          Hide
          reiss David Reiss added a comment -

          Sorry for the delay! I have made a few comments on the `pipe_tasks` PR; otherwise I think it is good to go, and nice to see this useful documentation.

          Show
          reiss David Reiss added a comment - Sorry for the delay! I have made a few comments on the `pipe_tasks` PR; otherwise I think it is good to go, and nice to see this useful documentation.

            People

            • Assignee:
              yusra Yusra AlSayyad
              Reporter:
              swinbank John Swinbank
              Reviewers:
              David Reiss
              Watchers:
              David Reiss, John Swinbank, Yusra AlSayyad
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel