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

Save each step of the AOS WEP algorithm for debugging purposes

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ts_aos
    • Labels:
    • Team:
      Telescope and Site
    • Urgent?:
      No

      Description

      The AOS WEP algorithm iterates through many steps, transforming donut images and masks. I will add a log to the algorithm that saves the images and masks at each step of the algorithm. This will be useful for understanding and debugging the algorithm.

        Attachments

          Issue Links

          contains

          Improvement - An improvement or enhancement to an existing feature or task. DM-36147 Pass convergence information in ts_wep

          • Done
          relates to

          Improvement - An improvement or enhancement to an existing feature or task. DM-39215 Align SITCOMTN-046 with the updated ts_wep algorithm

          • To Do

          Improvement - An improvement or enhancement to an existing feature or task. DM-37045 Check wavelength dependence in ts_aos

          • Done

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            jfc20 John Franklin Crenshaw created issue -
            jfc20 John Franklin Crenshaw made changes -
            Field Original Value New Value
            Status To Do [ 10001 ] In Progress [ 3 ]
            jfc20 John Franklin Crenshaw made changes -
            Labels UW
            Hide
            Permalink
            ttsai Te-Wei Tsai added a comment -

            It might be good to have a balance of log level and details.

            Show
            ttsai Te-Wei Tsai added a comment - It might be good to have a balance of log level and details.
            tjenness Tim Jenness made changes -
            Team Telescope and Site [ 13500 ]
            ksuberlak Krzysztof Suberlak made changes -
            Link This issue contains DM-36147 [ DM-36147 ]
            jfc20 John Franklin Crenshaw made changes -
            Reviewers Bryce Kalmbach, Krzysztof Suberlak, Te-Wei Tsai [ jbkalmbach, ksuberlak, ttsai ]
            jfc20 John Franklin Crenshaw made changes -
            Status In Progress [ 3 ] In Review [ 10004 ]
            Hide
            Permalink
            ttsai Te-Wei Tsai added a comment - - edited

            I reviewed the PR in GitHub. The questions may need Chris's and Bryce's inputs.

            Show
            ttsai Te-Wei Tsai added a comment - - edited I reviewed the PR in GitHub. The questions may need Chris's and Bryce's inputs.
            ksuberlak Krzysztof Suberlak made changes -
            Link This issue relates to DM-37045 [ DM-37045 ]
            Hide
            Permalink
            jfc20 John Franklin Crenshaw added a comment -

            Was reviewed extensively by Krzysztof Suberlak, Bryce Kalmbach, and Te-Wei Tsai on Github:  https://github.com/lsst-ts/ts_wep/pull/152

            Show
            jfc20 John Franklin Crenshaw added a comment - Was reviewed extensively by Krzysztof Suberlak ,  Bryce Kalmbach , and Te-Wei Tsai  on Github:   https://github.com/lsst-ts/ts_wep/pull/152
            jfc20 John Franklin Crenshaw made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            jfc20 John Franklin Crenshaw made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            ksuberlak Krzysztof Suberlak made changes -
            Link This issue relates to DM-39215 [ DM-39215 ]

              People

              Assignee:
              jfc20 John Franklin Crenshaw
              Reporter:
              jfc20 John Franklin Crenshaw
              Reviewers:
              Bryce Kalmbach, Krzysztof Suberlak, Te-Wei Tsai
              Watchers:
              Bryce Kalmbach, John Franklin Crenshaw, Krzysztof Suberlak, Te-Wei Tsai
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.