Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-577

Reduce FitTanSipWcsTask default order to 2

    XMLWordPrintable

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: meas_astrom
    • Labels:
      None

      Description

      The default SIP polynomial order for FitTanSipWcsTask is currently 4. This RFC proposes reducing the default to 2.

      Background:
      This fitter is used by processCcd.calibrate.astrometry (AstrometryTask) to get a first guess of wcs which is stored in the calexps. In a normal DRP, this wcs gets refined later in the jointcal step. Currently the fit is done per CCD and has no knowledge of the camera model. (We plan on doing full-focal plane fits in the future). AstrometryTask iterates through alternating matching and fitting steps. The default matcher was recently changed to the more robust MatchPessimisticB, but getting a robust WCS depends on both the matcher and fitter being robust.

      Reasons for the proposal:

      • Lower order is more robust
      • The default order of 4 has surprised some users. Most recently, in the DC2/imsim reprocessing, we've seen some failures due to bad WCS solutions which are prevented by a lower default order (e.g. DM-17737/DM-17731).

      obs_package plan to produce no change in behavior for existing cameras:

        Attachments

          Issue Links

            Activity

            Hide
            Parejkoj John Parejko added a comment -

            Somewhat surprised that there haven't been any comments on this so far.

            I think this is a reasonable proposal. The most important caveat that I can think of is how strong the optical distortions are on edge chips. Certainly, from the jointcal HSC fits, the WCS looks like it changes quite quickly toward the edges. I would worry that a 2nd order polynomial wouldn't be enough to handle that. I suppose that changing the obs packages to keep their current behavior means we can study it better in the future.

            A 2nd order fit is probably what we want anyway, once we start including the camera model.

            Show
            Parejkoj John Parejko added a comment - Somewhat surprised that there haven't been any comments on this so far. I think this is a reasonable proposal. The most important caveat that I can think of is how strong the optical distortions are on edge chips. Certainly, from the jointcal HSC fits, the WCS looks like it changes quite quickly toward the edges. I would worry that a 2nd order polynomial wouldn't be enough to handle that. I suppose that changing the obs packages to keep their current behavior means we can study it better in the future. A 2nd order fit is probably what we want anyway, once we start including the camera model.
            Hide
            yusra Yusra AlSayyad added a comment -

            Thanks for the comment John Parejko.  I agree. 

            All behavior for existing cameras will stay the same. Users processing data from new cameras will have to intentionally increase the order if they find they need it vs. intentionally decrease it when they get wcs fitting failures.  

            Show
            yusra Yusra AlSayyad added a comment - Thanks for the comment John Parejko .  I agree.  All behavior for existing cameras will stay the same. Users processing data from new cameras will have to intentionally increase the order if they find they need it vs. intentionally decrease it when they get wcs fitting failures.  

              People

              Assignee:
              yusra Yusra AlSayyad
              Reporter:
              yusra Yusra AlSayyad
              Watchers:
              Christopher Waters, John Parejko, John Swinbank, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                  Jenkins

                  No builds found.