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

fix hexapod GUI telemetry plots

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Sprint:
      TSSW Sprint - Apr 12 - Apr 26, TSSW Sprint - Apr 26 - May 10
    • Team:
      Telescope and Site
    • Urgent?:
      No

      Description

      The hexapod operator's menu, figure 16, (attached below), shows the telemetry plots. 

      However, it is not clear what the range of the time axis is, and in what unit.

      Hexapod tests on the summit conducted in November 2020 confirmed that the above figure agrees with the latest version of the software.

      To make this useful, we need to 

      (1) look into the unit and range of the time axis, and label them clearly.

      (2) add a drop down menu, so that users can choose "past 2 minutes", "past 5 minutes" etc to easily reconfigure the range of the time axis. (exact menu items TBD)

        Attachments

          Issue Links

            Activity

            No builds found.
            bxin Bo Xin [X] (Inactive) created issue -
            bxin Bo Xin [X] (Inactive) made changes -
            Field Original Value New Value
            Epic Link DM-27621 [ 441885 ]
            ttsai Te-Wei Tsai made changes -
            Story Points 1
            Labels HexRot MainTelescope ccw-rotator-integration-test
            ttsai Te-Wei Tsai made changes -
            Link This issue relates to DM-27887 [ DM-27887 ]
            ttsai Te-Wei Tsai made changes -
            End date 13/Apr/21
            Sprint TSSW Sprint - Apr 12 - Apr 26 [ 1089 ]
            Start date 12/Apr/21
            wvreeven Wouter van Reeven made changes -
            Sprint TSSW Sprint - Apr 12 - Apr 26 [ 1089 ] TSSW Sprint - Apr 12 - Apr 26, TSSW Sprint - Apr 26 - May 10 [ 1089, 1091 ]
            ttsai Te-Wei Tsai made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            Hide
            ttsai Te-Wei Tsai added a comment - - edited

            It looks like MOOG developers used the waveform graph instead of chart here. I could not reuse the vis in DM-27887.

            Added the Config class.

            Show
            ttsai Te-Wei Tsai added a comment - - edited It looks like MOOG developers used the waveform graph instead of chart here. I could not reuse the vis in DM-27887 . Added the Config class.
            Hide
            ttsai Te-Wei Tsai added a comment - - edited

            Updated the code to use the Config class. I realized the FGV is used to hold the data by the MOOG developers. After some tests, I believe the use of FGV is a bad idea.

            Removed the FGV and used the waveform chart instead of graph. The test with fake data:

            Show
            ttsai Te-Wei Tsai added a comment - - edited Updated the code to use the Config  class. I realized the FGV is used to hold the data by the MOOG developers. After some tests, I believe the use of FGV is a bad idea. Removed the FGV and used the waveform chart instead of graph. The test with fake data:
            ttsai Te-Wei Tsai made changes -
            Attachment fakeDataTest.png [ 49377 ]
            ttsai Te-Wei Tsai made changes -
            Story Points 1 2
            Hide
            ttsai Te-Wei Tsai added a comment -
            Show
            ttsai Te-Wei Tsai added a comment - Please review the PR: https://github.com/lsst-ts/ts_hexapod_gui/pull/8 Thanks!
            ttsai Te-Wei Tsai made changes -
            Reviewers Bo Xin [ bxin ] Bo Xin, Dave Mills [ bxin, dmills ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            ttsai Te-Wei Tsai made changes -
            Story Points 2 1
            Hide
            dmills Dave Mills added a comment -

            ok

            Show
            dmills Dave Mills added a comment - ok
            dmills Dave Mills made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            ttsai Te-Wei Tsai made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              People

              Assignee:
              ttsai Te-Wei Tsai
              Reporter:
              bxin Bo Xin [X] (Inactive)
              Reviewers:
              Bo Xin [X] (Inactive), Dave Mills
              Watchers:
              Bo Xin [X] (Inactive), Dave Mills, Te-Wei Tsai
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Start date:
                End date:

                  Jenkins

                  No builds found.