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

Publish the Velocity or Needed Telemetry to DDS for Rotator

    XMLWordPrintable

    Details

      Description

      Publish the velocity or needed telemetry to DDS for rotator.

        Attachments

        1. data.mat
          333 kB
        2. pt2ptMovePosNegative.png
          pt2ptMovePosNegative.png
          84 kB
        3. pt2ptMovePosPositive.png
          pt2ptMovePosPositive.png
          81 kB
        4. pt2ptMoveRateNegative.png
          pt2ptMoveRateNegative.png
          61 kB
        5. pt2ptMoveRatePositive.png
          pt2ptMoveRatePositive.png
          66 kB

          Issue Links

            Activity

            No builds found.
            ttsai Te-Wei Tsai created issue -
            ttsai Te-Wei Tsai made changes -
            Field Original Value New Value
            Epic Link DM-25674 [ 436565 ]
            ttsai Te-Wei Tsai made changes -
            Sprint TSSW Sprint - Jul 20 - Aug 3 [ 1034 ]
            ttsai Te-Wei Tsai made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            ttsai Te-Wei Tsai made changes -
            Attachment pt2ptMovePosPositive.png [ 44940 ]
            Attachment pt2ptMoveRatePositive.png [ 44941 ]
            Hide
            ttsai Te-Wei Tsai added a comment - - edited

            Do the point-to-point movement in the positive direction and check the available velocity telemetry as the following:

            It looks there is the problem of cha_rate_fb and chb_rate_fb. Therefore, I put the rateCmd to do the comparison as well.

            Put the negative direction below:

            The data of rate A and B are written in 'rotator_smlink_controller' -> 'Control Laws & Profile Generators (4 khz)' -> 'Controller' -> 'Channel-A/B velocity loop' subsystem.

            Show
            ttsai Te-Wei Tsai added a comment - - edited Do the point-to-point movement in the positive direction and check the available velocity telemetry as the following: It looks there is the problem of cha_rate_fb and chb_rate_fb . Therefore, I put the rateCmd to do the comparison as well. Put the negative direction below: The data of rate A and B are written in 'rotator_smlink_controller' -> 'Control Laws & Profile Generators (4 khz)' -> 'Controller' -> 'Channel-A/B velocity loop' subsystem.
            ttsai Te-Wei Tsai made changes -
            Attachment pt2ptMoveRateNegative.png [ 44942 ]
            Attachment pt2ptMovePosNegative.png [ 44943 ]
            ttsai Te-Wei Tsai made changes -
            Story Points 2 3
            Hide
            ttsai Te-Wei Tsai added a comment -

            Report the telemetry of velocity and no new track comand to high-level rotator CSC.

            Show
            ttsai Te-Wei Tsai added a comment - Report the telemetry of velocity and no new track comand to high-level rotator CSC.
            ttsai Te-Wei Tsai made changes -
            Attachment data.mat [ 45024 ]
            Hide
            ttsai Te-Wei Tsai added a comment -

            Add the raw data of Simulink model for the possible test in the high-level rotator CSC: data.mat.

            Show
            ttsai Te-Wei Tsai added a comment - Add the raw data of Simulink model for the possible test in the high-level rotator CSC:  data.mat .
            Hide
            ttsai Te-Wei Tsai added a comment -

            Please help to review the PR:
            https://github.com/lsst-ts/ts_rotator_controller/pull/9

            This PR will break the rotator CSC for the updated DDS telemetry. However, the initial test on summit will still use the original PXI code. That means the update of rotator CSC is not urgent.

            Thanks!

            Show
            ttsai Te-Wei Tsai added a comment - Please help to review the PR: https://github.com/lsst-ts/ts_rotator_controller/pull/9 This PR will break the rotator CSC for the updated DDS telemetry. However, the initial test on summit will still use the original PXI code. That means the update of rotator CSC is not urgent. Thanks!
            ttsai Te-Wei Tsai made changes -
            Reviewers Russell Owen [ rowen ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            rowen Russell Owen made changes -
            Link This issue is triggering DM-26107 [ DM-26107 ]
            Hide
            rowen Russell Owen added a comment -

            This looks very nice. I have a few small suggestions on github.

            Show
            rowen Russell Owen added a comment - This looks very nice. I have a few small suggestions on github.
            rowen Russell Owen made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            ttsai Te-Wei Tsai made changes -
            Link This issue relates to DM-26135 [ DM-26135 ]
            ttsai Te-Wei Tsai made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            rowen Russell Owen made changes -
            Link This issue relates to DM-26382 [ DM-26382 ]

              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:

                  CI Builds

                  No builds found.