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

Fix rotation direction of CWFS sensors

    XMLWordPrintable

    Details

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

      Description

      The rotation in the CWFS sensors in ts_wep is the wrong direction so the R04 and R40 sensors are 180 degrees off when estimating Zernikes.

        Attachments

        1. fwhmIters.png
          fwhmIters.png
          37 kB
        2. fwhmIters-1.png
          fwhmIters-1.png
          21 kB
        3. wep_v1.7.10.png
          wep_v1.7.10.png
          116 kB
        4. wep_v2.3.6.png
          wep_v2.3.6.png
          117 kB

          Activity

          Hide
          jbkalmbach Bryce Kalmbach added a comment -

          When using the original code and running the closed loop at high density of the wavefront sensors with 10 iterations the loop does not converge:

          Show
          jbkalmbach Bryce Kalmbach added a comment - When using the original code and running the closed loop at high density of the wavefront sensors with 10 iterations the loop does not converge:
          Hide
          jbkalmbach Bryce Kalmbach added a comment -

          After the change the loop converges:

          Show
          jbkalmbach Bryce Kalmbach added a comment - After the change the loop converges:
          Hide
          jbkalmbach Bryce Kalmbach added a comment - - edited

          Some more evidence that this rotation change is correct comes from rerunning our data from https://github.com/lsst-ts/ts_analysis_notebooks/blob/develop/aos/aos_milestones/lsstCwfsPipeline.ipynb.

           

          We can see that the new version (2.3.6) of ts_wep aligns better with the OPD values.

          Show
          jbkalmbach Bryce Kalmbach added a comment - - edited Some more evidence that this rotation change is correct comes from rerunning our data from https://github.com/lsst-ts/ts_analysis_notebooks/blob/develop/aos/aos_milestones/lsstCwfsPipeline.ipynb.   We can see that the new version (2.3.6) of ts_wep aligns better with the OPD values.

            People

            Assignee:
            jbkalmbach Bryce Kalmbach
            Reporter:
            jbkalmbach Bryce Kalmbach
            Watchers:
            Bryce Kalmbach
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

              Due:
              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.