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

Have the Rotator Send Exactly the Same Telemetry and Config Data to the EUI and CSC

    XMLWordPrintable

    Details

    • Story Points:
      3
    • Epic Link:
    • Sprint:
      TSSW Sprint - Jul 19 - Aug 02, TSSW Sprint - Aug 02 - Aug 16
    • Team:
      Telescope and Site
    • Urgent?:
      No

      Description

      Have the rotator send exactly the same telemetry and config data to the EUI and CSC. The details can follow DM-30811. This task will support to read the binary file of rotator EUI as well. This is for the debugging use only. The implementation can use the DM-30715 as a reference.

       
      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.

      Note. Update the rotator to use the thread-safe command buffer as well in this ticket as in DM-28687.

      Note. I realized this ticket will change the GUI for the structure size. Need to update the GUI for this.

        Attachments

        1. enable.png
          enable.png
          607 kB
        2. offline.png
          offline.png
          597 kB

          Issue Links

          is triggered by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-30811 Please have the hexapod 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-31230 Update MTRotator CSC for new data from low-level controller

          • Done
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-28687 The MTHexapod low-level controller mis-handles clearError if there is a fault condition

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-29721 Support the Command Queue in Low-Level Rotator Controller

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-30715 Rewrite the Logic to Assign the DDS Telemetry Data

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-29721 Support the Command Queue in Low-Level Rotator Controller

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-30550 Camera Hexapod Chronograph page presents jumping "calibrated actuator position" values.

          • Done

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            ttsai Te-Wei Tsai created issue -
            ttsai Te-Wei Tsai made changes -
            Field Original Value New Value
            Link This issue is triggered by DM-30811 [ DM-30811 ]
            ttsai Te-Wei Tsai made changes -
            Description Have the rotator send exactly the same telemetry and config data to the EUI and CSC. The details can follow DM-30811. Have the rotator send exactly the same telemetry and config data to the EUI and CSC. The details can follow DM-30811. This task will support to read the binary file of rotator EUI as well. This is for the debugging use only. The implementation can use the DM-30715 as a reference.
            ttsai Te-Wei Tsai made changes -
            Link This issue relates to DM-30715 [ DM-30715 ]
            ttsai Te-Wei Tsai made changes -
            Epic Link DM-27619 [ 441883 ] DM-27620 [ 441884 ]
            ttsai Te-Wei Tsai made changes -
            Description Have the rotator send exactly the same telemetry and config data to the EUI and CSC. The details can follow DM-30811. This task will support to read the binary file of rotator EUI as well. This is for the debugging use only. The implementation can use the DM-30715 as a reference. Have the rotator send exactly the same telemetry and config data to the EUI and CSC. The details can follow DM-30811. This task will support to read the binary file of rotator EUI as well. This is for the debugging use only. The implementation can use the DM-30715 as a reference.

             
            {color:#de350b}*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.*{color}
            hdrass Holger Drass made changes -
            Link This issue relates to DM-30550 [ DM-30550 ]
            hdrass Holger Drass made changes -
            Link This issue relates to SUMMIT-5251 [ SUMMIT-5251 ]
            hdrass Holger Drass made changes -
            Link This issue relates to SUMMIT-5404 [ SUMMIT-5404 ]
            hdrass Holger Drass made changes -
            Link This issue relates to SUMMIT-5404 [ SUMMIT-5404 ]
            ttsai Te-Wei Tsai made changes -
            Story Points 2
            ttsai Te-Wei Tsai made changes -
            Sprint TSSW Sprint - Jul 19 - Aug 02 [ 1108 ]
            ttsai Te-Wei Tsai made changes -
            Description Have the rotator send exactly the same telemetry and config data to the EUI and CSC. The details can follow DM-30811. This task will support to read the binary file of rotator EUI as well. This is for the debugging use only. The implementation can use the DM-30715 as a reference.

             
            {color:#de350b}*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.*{color}             		Have the rotator send exactly the same telemetry and config data to the EUI and CSC. The details can follow DM-30811. This task will support to read the binary file of rotator EUI as well. This is for the debugging use only. The implementation can use the DM-30715 as a reference.

             
            {color:#de350b}*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.*{color}

            Note. Update the rotator to use the thread-safe command buffer as well in this ticket as in DM-28687.
            ttsai Te-Wei Tsai made changes -
            Link This issue relates to DM-28687 [ DM-28687 ]
            rowen Russell Owen made changes -
            Link This issue is triggering DM-31230 [ DM-31230 ]
            ttsai Te-Wei Tsai made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            ttsai Te-Wei Tsai made changes -
            Description Have the rotator send exactly the same telemetry and config data to the EUI and CSC. The details can follow DM-30811. This task will support to read the binary file of rotator EUI as well. This is for the debugging use only. The implementation can use the DM-30715 as a reference.

             
            {color:#de350b}*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.*{color}

            Note. Update the rotator to use the thread-safe command buffer as well in this ticket as in DM-28687.             		Have the rotator send exactly the same telemetry and config data to the EUI and CSC. The details can follow DM-30811. This task will support to read the binary file of rotator EUI as well. This is for the debugging use only. The implementation can use the DM-30715 as a reference.

             
             {color:#de350b}*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.*{color}

            Note. Update the rotator to use the thread-safe command buffer as well in this ticket as in DM-28687.

            Note. I realized this ticket will change the GUI for the structure size. Need to update the GUI for this.
            ttsai Te-Wei Tsai made changes -
            Link This issue relates to DM-29721 [ DM-29721 ]
            ttsai Te-Wei Tsai made changes -
            Link This issue relates to DM-29721 [ DM-29721 ]
            Hide
            Permalink
            ttsai Te-Wei Tsai added a comment -

            The thread-safe queue was done in DM-29721 already.

            Show
            ttsai Te-Wei Tsai added a comment - The thread-safe queue was done in DM-29721 already.
            Hide
            Permalink
            ttsai Te-Wei Tsai added a comment -

            Finished the update of ts_rotator_controller.

            Show
            ttsai Te-Wei Tsai added a comment - Finished the update of ts_rotator_controller .
            wvreeven Wouter van Reeven made changes -
            Sprint TSSW Sprint - Jul 19 - Aug 02 [ 1108 ] TSSW Sprint - Jul 19 - Aug 02, TSSW Sprint - Aug 02 - Aug 16 [ 1108, 1111 ]
            ttsai Te-Wei Tsai made changes -
            Story Points 2 3
            Hide
            Permalink
            ttsai Te-Wei Tsai added a comment - - edited

            Updated the GUI for the frame size and reformat the front panel. Removed the command frame. Supported the reading of binary file.

            Show
            ttsai Te-Wei Tsai added a comment - - edited Updated the GUI for the frame size and reformat the front panel. Removed the command frame. Supported the reading of binary file.
            ttsai Te-Wei Tsai made changes -
            Attachment enable.png [ 51472 ]
            Attachment offline.png [ 51473 ]
            Hide
            Permalink
            ttsai Te-Wei Tsai added a comment -

            Tested the updated code on summit to make sure I can get the configuration data:
                 

            Show
            ttsai Te-Wei Tsai added a comment - Tested the updated code on summit to make sure I can get the configuration data:      
            Hide
            Permalink
            ttsai Te-Wei Tsai added a comment -

            Please help to review the PRs:
            1. https://github.com/lsst-ts/ts_rotator_controller/pull/24
            2. https://github.com/lsst-ts/ts_rotator_gui/pull/13

            Thanks!

            Show
            ttsai Te-Wei Tsai added a comment - Please help to review the PRs: 1. https://github.com/lsst-ts/ts_rotator_controller/pull/24 2. https://github.com/lsst-ts/ts_rotator_gui/pull/13 Thanks!
            ttsai Te-Wei Tsai made changes -
            Reviewers Russell Owen [ rowen ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            Hide
            Permalink
            rowen Russell Owen added a comment - - edited

            Reviewed on github. Thank you for a very useful improvement and for making sure all the telemetry that the CSC uses was being output.

            Show
            rowen Russell Owen added a comment - - edited Reviewed on github. Thank you for a very useful improvement and for making sure all the telemetry that the CSC uses was being output.
            rowen Russell Owen 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:
              ttsai Te-Wei Tsai
              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.