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

Align phosim_syseng4 rotation to obs_lsst

    XMLWordPrintable

    Details

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

      Description

      Phosim_syseng4  orientation of the CCDs in the focal plane is offset by 90 degrees with respect to obs_lsst. This means that images simulated with phosim with rotSkyPos of 0 need to be ingested into a Gen3 repo with ROTANG of 90 for proper WCS alignment, or handled within `ts_phosim` (see https://github.com/lsst-dm/phosim_utils/pull/19).   The offset is described eg, https://github.com/lsst-ts/phosim_syseng4/pull/6 and https://github.com/lsst-ts/phosim_syseng4/pull/7 .  This ticket is to fix that offset by aligning phosim orientation with that of obs_lsst, for {{segmentation.txt }}and {{focalplanelayout.txt in  }}data/comcam and data/lsst 

       

      Completion criteria

      • ensure that phosim raw files can be repackaged, ingested, and  correctly ISR in a gen3 repo
      • run imgCloseLoop and opdCloseLoop before and after the change, ensure that the loop converges

        Attachments

          Issue Links

            Activity

            Hide
            ksuberlak Krzysztof Suberlak added a comment -

            A notebook describing the summary of the work done to update the focalplanelayout is at https://github.com/suberlak/AOS/blob/main/AOS_DM-30367_summary.ipynb

            Show
            ksuberlak Krzysztof Suberlak added a comment - A notebook describing the summary of the work done to update the focalplanelayout is at https://github.com/suberlak/AOS/blob/main/AOS_DM-30367_summary.ipynb
            Hide
            bxin Bo Xin [X] (Inactive) added a comment -

            Notebook is very nice. Thanks for getting to the bottom of this!

            Show
            bxin Bo Xin [X] (Inactive) added a comment - Notebook is very nice. Thanks for getting to the bottom of this!
            Hide
            ksuberlak Krzysztof Suberlak added a comment -

            Merged https://github.com/lsst-ts/ts_wep/pull/98 , this completes the changes necessitated by this ticket

            Show
            ksuberlak Krzysztof Suberlak added a comment - Merged https://github.com/lsst-ts/ts_wep/pull/98 , this completes the changes necessitated by this ticket
            Hide
            ttsai Te-Wei Tsai added a comment -

            Reviewed the code in Github. Please help to update the doc/versionHistory.rst of ts_wep

            Show
            ttsai Te-Wei Tsai added a comment - Reviewed the code in Github. Please help to update the doc/versionHistory.rst of ts_wep
            Hide
            ksuberlak Krzysztof Suberlak added a comment -

            Updated the version history

            Show
            ksuberlak Krzysztof Suberlak added a comment - Updated the version history

              People

              Assignee:
              ksuberlak Krzysztof Suberlak
              Reporter:
              ksuberlak Krzysztof Suberlak
              Reviewers:
              Te-Wei Tsai
              Watchers:
              Bo Xin [X] (Inactive), Krzysztof Suberlak, Te-Wei Tsai
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Start date:

                  Jenkins

                  No builds found.