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

Single amp reads for obs_lsst have the wrong geometry.

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: obs_lsst
    • Labels:
      None

      Description

      As reported by Eric Charles, single amp reads for ITL detectors fails to return the correct bounding boxes.  In the given example (see attached script), amplifier C13 claims to have rawDataBBox = Box2I(minimum=Point2I(99, 0), dimensions=Extent2I(509, 2000)), however the full image area isn't this large: getBBox = Box2I(minimum=Point2I(0, 0), dimensions=Extent2I(576, 2048))

        Attachments

          Activity

          Hide
          czw Christopher Waters added a comment -

          I think I've tracked this down.  The process of updating the overscan regions also updates the XY offset for each amplifier.  The current code transforms the to-be-used amplifier according to the input amplifier offsets, accumulating an ix*NadditionalPixels error.  By transforming against the newly created XY offsets, this error is removed.

          Show
          czw Christopher Waters added a comment - I think I've tracked this down.  The process of updating the overscan regions also updates the XY offset for each amplifier.  The current code transforms the to-be-used amplifier according to the input amplifier offsets, accumulating an ix*NadditionalPixels error.  By transforming against the newly created XY offsets, this error is removed.
          Hide
          jbosch Jim Bosch added a comment -

          I am worried that this fix means things that other new-ish things in cameraGeom are now broken; I'd like to resolve that before signing off.

          Show
          jbosch Jim Bosch added a comment - I am worried that this fix means things that other new-ish things in cameraGeom are now broken; I'd like to resolve that before signing off.

            People

            Assignee:
            czw Christopher Waters
            Reporter:
            czw Christopher Waters
            Reviewers:
            Jim Bosch
            Watchers:
            Christopher Waters, Eric Charles, Jim Bosch, Robert Lupton
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins Builds

                No builds found.