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

CalibrateTask should optionally not compute initial astrometry

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Invalid
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_tasks
    • Labels:
      None

      Description

      For processing data which already has a good WCS (such as CFHT and SDSS) it is unnecessary to perform an initial fit of astrometry before computing the PSF. Furthermore it is risky to do so because the astrometry fitter may fail.

      At present CFHT handles this by overriding the CalibrateTask with its own variant task and a variant run method. However, the amount of code duplication required is unacceptable.

      To avoid this I suggest we add the ability to disable the initial astrometric fit.

      I also think we should Refactor CalibrateTask so its run method is much simpler and easier to override. But even if we do that (and perhaps it should be a separate ticket) it is such a common need to not run the initial astrometry that I think one should not have to override the run method to do it.

        Attachments

          Issue Links

            Activity

            Hide
            krughoff Simon Krughoff added a comment -

            This came up as one of the requirements for the ProcessCcdTask. Rather than have that work in two places, we are putting the work for refactoring that task in one story, DM-4692.

            Show
            krughoff Simon Krughoff added a comment - This came up as one of the requirements for the ProcessCcdTask. Rather than have that work in two places, we are putting the work for refactoring that task in one story, DM-4692 .
            Hide
            swinbank John Swinbank added a comment -

            We anticipate a significant redesign of the major pipeline tasks – which would include this – in the Winter 2016 cycle. See DLP-495 & DLP-496 for details.

            Show
            swinbank John Swinbank added a comment - We anticipate a significant redesign of the major pipeline tasks – which would include this – in the Winter 2016 cycle. See DLP-495 & DLP-496 for details.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              rowen Russell Owen
              Watchers:
              Dominique Boutigny, Jim Bosch, John Swinbank, Paul Price, Russell Owen, Simon Krughoff, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  CI Builds

                  No builds found.