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

Astrometry matcher needs to load reference objects relative to max shift/rotation

    XMLWordPrintable

Details

    Description

      Currently the catalog of reference objects in the matcher is not loaded on a area relative to the max shift/rotation allowed. For larger shifts/rotations/distortions this could lead to portions of the chip being outside of the loaded reference catalog area and thus unconstrained in future wcs fitting.

      Deliverables: Fix to meas_astrom so that it loads an area of the reference catalog relative to the max shift/rotation.

      Attachments

        Activity

          No builds found.
          cmorrison Chris Morrison [X] (Inactive) created issue -
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Field Original Value New Value
          Epic Link DM-7366 [ 26452 ]
          price Paul Price added a comment -

          Instead of adding more I/O, we should load the catalog once only with an additional buffer in the search radius, where the buffer is equal to the maximum possible offset.

          price Paul Price added a comment - Instead of adding more I/O, we should load the catalog once only with an additional buffer in the search radius, where the buffer is equal to the maximum possible offset.

          I agree with price; the reference object loader already grows the region by some amount to help with this (as per Paul's suggestion). I don't think it's directly tied to the maximum shift distance though---I vaguely remember it being a fixed size in pixels? It might make more sense to just update that logic.

          ctslater Colin Slater added a comment - I agree with price ; the reference object loader already grows the region by some amount to help with this (as per Paul's suggestion). I don't think it's directly tied to the maximum shift distance though---I vaguely remember it being a fixed size in pixels? It might make more sense to just update that logic.

          Okay I'll change the title and description.

          cmorrison Chris Morrison [X] (Inactive) added a comment - Okay I'll change the title and description.
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Summary Astrometry matcher needs to re-load refence objects after shift/rotation Astrometry matcher needs to load reference objects relative to max shift/rotation
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Description Currently the catalog of reference objects in the matcher is not reloaded after the wcs is corrected. For larger shifts/rotations/distortions this could lead to portions of the chip being outside of the loaded reference catalog area and thus unconstrained in future wcs fitting.

          Deliverables: Fix to meas_astrom that reloads or adds to the reference catalog after the wcs is updated.
          Currently the catalog of reference objects in the matcher is not loaded on a area relative to the max shift/rotation allowed. For larger shifts/rotations/distortions this could lead to portions of the chip being outside of the loaded reference catalog area and thus unconstrained in future wcs fitting.

          Deliverables: Fix to meas_astrom so that it loads an area of the reference catalog relative to the max shift/rotation.

          Found how to add a static padding to the bounding box used to load the reference catalog. Need to tie this padding amount to the maximum allowed shift in the matcher.

          cmorrison Chris Morrison [X] (Inactive) added a comment - Found how to add a static padding to the bounding box used to load the reference catalog. Need to tie this padding amount to the maximum allowed shift in the matcher.
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Status To Do [ 10001 ] In Progress [ 3 ]

          The work of this ticket was completed inside of DM-9750.

          cmorrison Chris Morrison [X] (Inactive) added a comment - The work of this ticket was completed inside of DM-9750 .
          cmorrison Chris Morrison [X] (Inactive) made changes -
          Resolution Done [ 10000 ]
          Status In Progress [ 3 ] Invalid [ 11005 ]

          People

            cmorrison Chris Morrison [X] (Inactive)
            cmorrison Chris Morrison [X] (Inactive)
            Chris Morrison [X] (Inactive), Colin Slater, Paul Price
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Jenkins

                No builds found.