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

          Hide
          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.

          Show
          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.
          Hide
          ctslater Colin Slater added a comment -

          I agree with Paul 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.

          Show
          ctslater Colin Slater added a comment - I agree with Paul 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.
          Hide
          cmorrison Chris Morrison [X] (Inactive) added a comment -

          Okay I'll change the title and description.

          Show
          cmorrison Chris Morrison [X] (Inactive) added a comment - Okay I'll change the title and description.
          Hide
          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.

          Show
          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.
          Hide
          cmorrison Chris Morrison [X] (Inactive) added a comment -

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

          Show
          cmorrison Chris Morrison [X] (Inactive) added a comment - The work of this ticket was completed inside of DM-9750 .

            People

            Assignee:
            cmorrison Chris Morrison [X] (Inactive)
            Reporter:
            cmorrison Chris Morrison [X] (Inactive)
            Watchers:
            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 Builds

                No builds found.