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

Set FitSipDistortionTask as default and remove FitTanSipWcsTask (etc)

    XMLWordPrintable

    Details

    • Story Points:
      3

      Description

      Once the new FitSipDistortionTask introduced on DM-3549 has been fully vetted, it should be set as the default. The task it replaces (FitTanSipWcsTask) and the C++ code it uses (namespace meas::astrom::sip) should then be removed.

        Attachments

          Issue Links

            Activity

            No builds found.
            jbosch Jim Bosch created issue -
            jbosch Jim Bosch made changes -
            Field Original Value New Value
            Link This issue relates to DM-3549 [ DM-3549 ]
            Hide
            jbosch Jim Bosch added a comment -

            Code comments about this plan should also be removed from fitSipDistortion.py.

            Show
            jbosch Jim Bosch added a comment - Code comments about this plan should also be removed from fitSipDistortion.py.
            Hide
            jbosch Jim Bosch added a comment -

            Removing the old SIP fitter will also require using the new one in the astrometry.net tasks (or removing them), so this is a bit trickier than I originally guessed.

            Show
            jbosch Jim Bosch added a comment - Removing the old SIP fitter will also require using the new one in the astrometry.net tasks (or removing them), so this is a bit trickier than I originally guessed.
            jbosch Jim Bosch made changes -
            Story Points 1 3
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 23885 ]
            Hide
            swinbank John Swinbank added a comment -

            Jim Bosch — thoughts about the current usefulness of this ticket?

            In particular, I note:

            • The astrometry.net tasks have been removed, so presumably no effort is required there.
            • I'm not clear if this is useful, or if we should just continue moving (slowly) towards using the new fitter on DM-20371.
            • Not setting this as default means that it has (presumably) never received the sort of battle-testing that would qualify it as “fully vetted”.
            Show
            swinbank John Swinbank added a comment - Jim Bosch — thoughts about the current usefulness of this ticket? In particular, I note: The astrometry.net tasks have been removed, so presumably no effort is required there. I'm not clear if this is useful, or if we should just continue moving (slowly) towards using the new fitter on DM-20371 . Not setting this as default means that it has (presumably) never received the sort of battle-testing that would qualify it as “fully vetted”.
            Hide
            jbosch Jim Bosch added a comment -

            We can probably drop this fitter and associated tickets in favor of the DM-20371 one.

            Show
            jbosch Jim Bosch added a comment - We can probably drop this fitter and associated tickets in favor of the DM-20371 one.
            Hide
            swinbank John Swinbank added a comment -

            Thanks Jim.

            Show
            swinbank John Swinbank added a comment - Thanks Jim.
            swinbank John Swinbank made changes -
            Resolution Done [ 10000 ]
            Status To Do [ 10001 ] Won't Fix [ 10405 ]

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              jbosch Jim Bosch
              Watchers:
              Jim Bosch, John Swinbank
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.