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
            No builds found.
            rowen Russell Owen created issue -
            rowen Russell Owen made changes -
            Field Original Value New Value
            Link This issue is triggered by DM-30814 [ DM-30814 ]
            rowen Russell Owen made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            rowen Russell Owen made changes -
            Story Points 2 1
            rowen Russell Owen made changes -
            Sprint TSSW Sprint - Aug 02 - Aug 16 [ 1111 ]
            rowen Russell Owen made changes -
            Description Update the MTRotator CSC for changes in the telemetry (and configuration?) data from the low-level controller in DM-30814 Update the MTRotator CSC for changes in the telemetry message from the low-level controller in DM-30814.

            Note: the configuration message is unchanged.
            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
            rowen Russell Owen made changes -
            Reviewers Te-Wei Tsai [ ttsai ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            aclements Andy Clements made changes -
            Assignee Russell Owen [ rowen ]
            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!
            ttsai Te-Wei Tsai made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            ttsai Te-Wei Tsai made changes -
            Link This issue is triggering DM-31383 [ DM-31383 ]
            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
            rowen Russell Owen made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              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.