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

Update MTRotator CSC for new data from low-level controller

    XMLWordPrintable

    Details

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

      Description

      Update the MTRotator CSC for changes in the telemetry message from the low-level controller in DM-30814.

      Note: the configuration message is unchanged.

        Attachments

          Issue Links

          is triggered by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-30814 Have the Rotator Send Exactly the Same Telemetry and Config Data to the EUI and CSC

          • Done
          is triggering

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-31383 Please document low-level MT rotator telemetry fields

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            rowen Russell Owen added a comment -

            I also changed the struct to use explicit bytes types (e.g. ctypes.c_uint32 instead of ctypes.c_uint). I had not seen those in the ctypes documentation before.

            Pull request: https://github.com/lsst-ts/ts_mtrotator/pull/47

            Show
            rowen Russell Owen added a comment - I also changed the struct to use explicit bytes types (e.g. ctypes.c_uint32 instead of ctypes.c_uint). I had not seen those in the ctypes documentation before. Pull request: https://github.com/lsst-ts/ts_mtrotator/pull/47
            Hide
            Permalink
            ttsai Te-Wei Tsai added a comment -

            The update looks good to me. I have some question related to the number of fields of Simulink telemetry. I thought there should be 45 fields (double values, Simulink parts) in your Telemetry but I only found 38. Thanks!

            Show
            ttsai Te-Wei Tsai added a comment - The update looks good to me. I have some question related to the number of fields of Simulink telemetry. I thought there should be 45 fields (double values, Simulink parts) in your Telemetry but I only found 38. Thanks!
            Hide
            Permalink
            rowen Russell Owen added a comment -

            I found the missing telemetry fields and checked the results. Te-Wei Tsai also checked the fields. We think it's right now.

            Released v0.16.0

            Show
            rowen Russell Owen added a comment - I found the missing telemetry fields and checked the results. Te-Wei Tsai also checked the fields. We think it's right now. Released v0.16.0

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Reviewers:
              Te-Wei Tsai
              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.