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

Implement known PSF and astrometry improvements in CalibrateImageTask

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • pipe_tasks
    • None
    • 2
    • AP S24-2 (January), AP S24-3 (February)
    • Alert Production
    • No

    Description

      While pair coding with Ian, I learned about some changes we should make to CalibrateImageTask to improve the PSF and astrometry residuals (which should help a bit with the excess dipole problem). This ticket is to implement those improvements.

      Changes include:

      • thresholdValue/includeThresholdMultiplier
      • ensuring PSF sources are actually isolated
      • reserving PSF/astrometry/photometry sources

      Attachments

        Issue Links

          Activity

            Parejkoj John Parejko added a comment - - edited

            sullivan: Mind doing this very small review? I only implemented three real changes here, but they should help.

            "Ensure PSF sources are isolated" is tricky, and the existing CalibrateTask isn't reserving
            astrometry/photometry sources right now, so we can deal with that we need it.

            PR: https://github.com/lsst/pipe_tasks/pull/889

            Jenkins: https://rubin-ci.slac.stanford.edu/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/678/pipeline

            Parejkoj John Parejko added a comment - - edited sullivan : Mind doing this very small review? I only implemented three real changes here, but they should help. "Ensure PSF sources are isolated" is tricky, and the existing CalibrateTask isn't reserving astrometry/photometry sources right now, so we can deal with that we need it. PR: https://github.com/lsst/pipe_tasks/pull/889 Jenkins: https://rubin-ci.slac.stanford.edu/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/678/pipeline

            The changes look straightforward, and should be improvements.

            sullivan Ian Sullivan added a comment - The changes look straightforward, and should be improvements.

            People

              Parejkoj John Parejko
              Parejkoj John Parejko
              Ian Sullivan
              Eric Bellm, Ian Sullivan, John Parejko
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.