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

More verbose bump test, fix wait for test progress

    XMLWordPrintable

    Details

      Description

      There are two issues with the test scripts, which deserve to fix:

      1. T002 still sometimes tries to execute a bump test when the previous one is in progress. This is as it sense test is complete/failed from the last execution. Using next with some timeout, similar to what is now done in hardpoint testing, shall fix the issue

      2. T018, bump test while raised, deserves detailed information about why it failed. It is very hard to find in a set of 250+ FA the one which triggered the failure (any FA outside of baseline values can trigger a failure..).

        Attachments

          Issue Links

            Activity

            Hide
            pkubanek Petr Kubanek added a comment -

            Fixed T002. Need to work on T018 changes.

            Show
            pkubanek Petr Kubanek added a comment - Fixed T002. Need to work on T018 changes.
            Hide
            pkubanek Petr Kubanek added a comment -

            It should run now alright, please approve after verifying:

            https://github.com/lsst-ts/ts_m1m3supporttesting/pull/23

            Show
            pkubanek Petr Kubanek added a comment - It should run now alright, please approve after verifying: https://github.com/lsst-ts/ts_m1m3supporttesting/pull/23
            Hide
            pkubanek Petr Kubanek added a comment -

            This is one the the T018 problems. It doesn't wait too long after mirror raised, and collect baseline when it's still oscillating, in middle of the oscillations (on otherwise flat curve):

            Show
            pkubanek Petr Kubanek added a comment - This is one the the T018 problems. It doesn't wait too long after mirror raised, and collect baseline when it's still oscillating, in middle of the oscillations (on otherwise flat curve):
            Hide
            pkubanek Petr Kubanek added a comment -

            We changed slope settings, and that affect slope convergence. So the chances are we catch oscillations in ~3 seconds from time from time we finish the previous bump test

            Show
            pkubanek Petr Kubanek added a comment - We changed slope settings, and that affect slope convergence. So the chances are we catch oscillations in ~3 seconds from time from time we finish the previous bump test
            Hide
            bquint Bruno Quint added a comment -

            The PR and Ticket are approved. But I believe the PR needs to be merged. 

            Show
            bquint Bruno Quint added a comment - The PR and Ticket are approved. But I believe the PR needs to be merged. 

              People

              Assignee:
              pkubanek Petr Kubanek
              Reporter:
              pkubanek Petr Kubanek
              Reviewers:
              Bruno Quint
              Watchers:
              Bruno Quint, HyeYun Park, Petr Kubanek, Yijung Kang
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.