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

Investigate image orientation from phosim

    XMLWordPrintable

    Details

    • Type: Story
    • Status: In Progress
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: obs_lsst
    • Labels:
      None

      Description

      The WFS team has brought up what appears to be an inconsistency in the orientation of the output of ISR as compared to the camera coordinate system. Specifically, there appears to be a transposition in the postIsrCcd as compared to the expected orientation.

      Specifically, we in data management have defined the detector coordinate system as having C10 coincide with the origin.

      The camera coordinate system as defined in LCA-13381 has C00 coincident with the origin.

      This implies a simple rotation between the two systems, however, in the repackaging phase, the amps are renamed making them inconsistent. This is an image of where phosim thinks C00 lands in the camera coordinate system:

      That seems consistent with the definition in LCA-13381 page 5 (note that in that document the origin is in the lower right of the chip). However, the renamed amps make that chip C10 which does not agree with that coordinate system.

      I don't think we should be renaming the amps and should fix the assembly accordingly.

        Attachments

        1. image (1).png
          image (1).png
          254 kB
        2. image-2019-01-11-13-41-25-113.png
          image-2019-01-11-13-41-25-113.png
          8 kB
        3. image-2019-01-11-13-44-13-515.png
          image-2019-01-11-13-44-13-515.png
          8 kB
        4. image-2019-01-11-13-49-22-807.png
          image-2019-01-11-13-49-22-807.png
          74 kB
        5. screenshot-1.png
          screenshot-1.png
          29 kB
        6. screenshot-2.png
          screenshot-2.png
          848 kB

          Issue Links

            Activity

            Hide
            krughoff Simon Krughoff added a comment -

            I have made changes to both obs_lsst and phosim_utils to try to get a handle on this. If I change the repackager to not rename channels, things work as I expect.

            Eimage from phosim which I've been assuming is in the camera definition coordinate system:

            With the version of code and policy in the ticket branch the postIsrCcd looks like this:

            As you can see, this is pure rotation and in the direction I expect given the diagrams in the description.

            Of course changing cameraHeader.yaml will cause problems assembling other camera's sensors, but that seems like that's the correct thing to do given my understanding of the coordinate systems involved.

            Merlin Fisher-Levine and Robert Lupton please let me know if you can see any mistakes I'm making.

            Show
            krughoff Simon Krughoff added a comment - I have made changes to both obs_lsst and phosim_utils to try to get a handle on this. If I change the repackager to not rename channels, things work as I expect. Eimage from phosim which I've been assuming is in the camera definition coordinate system: With the version of code and policy in the ticket branch the postIsrCcd looks like this: As you can see, this is pure rotation and in the direction I expect given the diagrams in the description. Of course changing cameraHeader.yaml will cause problems assembling other camera's sensors, but that seems like that's the correct thing to do given my understanding of the coordinate systems involved. Merlin Fisher-Levine and Robert Lupton please let me know if you can see any mistakes I'm making.

              People

              Assignee:
              krughoff Simon Krughoff
              Reporter:
              krughoff Simon Krughoff
              Watchers:
              John Parejko, Merlin Fisher-Levine, Robert Lupton, Simon Krughoff
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated: