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

Please have the hexapod send exactly the same telemetry and config data to the EUI and CSC

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Sprint:
      TSSW Sprint - Jul 19 - Aug 02
    • Team:
      Telescope and Site
    • Urgent?:
      No

      Description

      Please consider having the low-level MTHexapod controller send exactly the same telemetry and configuration packets to the CSC as it sends to the EUI/GUI, instead of having to maintain two separate structures that are so similar.

      I have tested the CSC and it can keep up with 2x the data it is getting now at 20 Hz. That said, we have no need for 20 Hz so I suggest you offer the ability to cut the rate of telemetry to the CSC, e.g. by skipping every other telemetry packet (as long as that won't miss any important state changes).

      If this also applies to the MTRotator then please consider filing a ticket for that.

      In my opinion this is not high priority, but is a "nice to have" in that it should make the low-level controller code a bit easier to maintain and understand. It will require small changes to the CSC; one of us can file a separate ticket for that once you figure out if you have time and inclination to implement this one.

      Note. For the configuration (or parameter) telemetry, it should be published only when there is the value change.

      Note. Rename the structure related to the parameter/config to have no word of telemetry, which is confusing. This change will affect the EUI as well.

        Attachments

          Issue Links

            Activity

            Hide
            ttsai Te-Wei Tsai added a comment - - edited

            DDS and GUI have the same telemetry now. Removed the unused telemetry structure. Renamed the telemetry of parameter structure.

            Updated the GUI for the rename of telemetry. Removed the command frame.

            Supported to configure the DDS telemetry rate.

            Show
            ttsai Te-Wei Tsai added a comment - - edited DDS and GUI have the same telemetry now. Removed the unused telemetry structure. Renamed the telemetry of parameter structure. Updated the GUI for the rename of telemetry. Removed the command frame. Supported to configure the DDS telemetry rate.
            Hide
            ttsai Te-Wei Tsai added a comment -

            Please help to review the PRs:
            1. https://github.com/lsst-ts/ts_hexapod_controller/pull/21
            2. https://github.com/lsst-ts/ts_hexapod_gui/pull/15

            I do not test these PRs on summit yet because I believe I may need your help to update the CSC to test this ticket.

            Thanks!

            Show
            ttsai Te-Wei Tsai added a comment - Please help to review the PRs: 1. https://github.com/lsst-ts/ts_hexapod_controller/pull/21 2. https://github.com/lsst-ts/ts_hexapod_gui/pull/15 I do not test these PRs on summit yet because I believe I may need your help to update the CSC to test this ticket. Thanks!
            Hide
            rowen Russell Owen added a comment -

            Reviewed on github

            Show
            rowen Russell Owen added a comment - Reviewed on github

              People

              Assignee:
              ttsai Te-Wei Tsai
              Reporter:
              rowen Russell Owen
              Reviewers:
              Russell Owen
              Watchers:
              Russell Owen, Te-Wei Tsai
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.