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

PhotoCal should not use MatchOptimisticBTask

    XMLWordPrintable

    Details

      Description

      PhotoCalTask currently inherits from lsst.meas.algorithms.RefMatchTask, which defaults to using the MatchOptimisticBTask for matching. The MatchOptimisticBTask is a full-featured matcher allowing for offsets and rotations, but by the time photometric calibration is required we are expected to have done all astrometric calibration and the use of a full-featured matcher is inviting trouble and requires unnecessary duplication of configuration changes (i.e., any configuration changes need to be mirrored from calibrate.astrometry.matcher to calibrate.photoCal.matcher).

      I suggest changing the PhotoCalTask to use the DirectMatchTask, in which case the inheritance from RefMatchTask could be removed, greatly simplifying the design.

        Attachments

          Issue Links

            Activity

            Hide
            price Paul Price added a comment -

            Duplicate of RFC-405/DM-12207.

            Show
            price Paul Price added a comment - Duplicate of RFC-405 / DM-12207 .

              People

              Assignee:
              krughoff Simon Krughoff
              Reporter:
              price Paul Price
              Watchers:
              John Swinbank, Paul Price
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.