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

Fix issues discovered in SLAC rotator testing

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Sprint:
      TSSW Sprint - Oct 28 - Nov 10, TSSW Sprint - Nov 11 - Nov 22
    • Team:
      Telescope and Site

      Description

      In SLAC rotator testing today we discovered several issues that need fixing:

      • The counter field was the wrong type for Command and Header (should be uint, not ushort).
      • The rotator and hexapod CSCs have no heartbeat output. This strongly suggests we should have a base class for both CSCs.
      • The rotator commander outputs too much noise because of encoder jitter (not a surprise). This is likely an issue for the hexapod commander as well.

      Note that we still were not able to command the rotator: the commands were ignored. Fixing that is a different ticket.

      I have attached a file that shows some binary telemetry data we read from the rotator, and how the Python code parses it.

        Attachments

          Issue Links

            Activity

            rowen Russell Owen created issue -
            rowen Russell Owen made changes -
            Field Original Value New Value
            Epic Link DM-21473 [ 424069 ]
            rowen Russell Owen made changes -
            Link This issue relates to DM-21951 [ DM-21951 ]
            rowen Russell Owen made changes -
            Description In SLAC rotator testing today we discovered several issues that need fixing:

            * The {{counter}} field was the wrong type for Command and Header (should be uint, not ushort).
            * The rotator and hexapod CSCs have no heartbeat output
            * The commanders output too much noise because of encoder jitter (not a surprise)

            Note that we still were not able to command the rotator: the commands were ignored. Fixing that is a different ticket.
            In SLAC rotator testing today we discovered several issues that need fixing:

            * The {{counter}} field was the wrong type for Command and Header (should be uint, not ushort).
            * The rotator and hexapod CSCs have no heartbeat output. This strongly suggests we should have a base class for both CSCs.
            * The rotator commander outputs too much noise because of encoder jitter (not a surprise). This is likely an issue for the hexapod commander as well.

            Note that we still were not able to command the rotator: the commands were ignored. Fixing that is a different ticket.
            rowen Russell Owen made changes -
            Link This issue relates to DM-22067 [ DM-22067 ]
            rowen Russell Owen made changes -
            Attachment rotator_binary_data.py [ 41020 ]
            rowen Russell Owen made changes -
            Description In SLAC rotator testing today we discovered several issues that need fixing:

            * The {{counter}} field was the wrong type for Command and Header (should be uint, not ushort).
            * The rotator and hexapod CSCs have no heartbeat output. This strongly suggests we should have a base class for both CSCs.
            * The rotator commander outputs too much noise because of encoder jitter (not a surprise). This is likely an issue for the hexapod commander as well.

            Note that we still were not able to command the rotator: the commands were ignored. Fixing that is a different ticket.
            In SLAC rotator testing today we discovered several issues that need fixing:

            * The {{counter}} field was the wrong type for Command and Header (should be uint, not ushort).
            * The rotator and hexapod CSCs have no heartbeat output. This strongly suggests we should have a base class for both CSCs.
            * The rotator commander outputs too much noise because of encoder jitter (not a surprise). This is likely an issue for the hexapod commander as well.

            Note that we still were not able to command the rotator: the commands were ignored. Fixing that is a different ticket.

            I have attached a file that shows some binary telemetry data we read from the rotator, and how the Python code parses it.
            rowen Russell Owen made changes -
            Link This issue relates to DM-22104 [ DM-22104 ]
            rowen Russell Owen made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            rowen Russell Owen made changes -
            Reviewers Te-Wei Tsai [ ttsai ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            ttsai Te-Wei Tsai made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            ttsai Te-Wei Tsai made changes -
            Link This issue relates to DM-22146 [ DM-22146 ]
            rowen Russell Owen made changes -
            Link This issue mitigates DM-22150 [ DM-22150 ]
            tribeiro Tiago Ribeiro made changes -
            Sprint TSSW Sprint - Oct 28 - Nov 10 [ 966 ] TSSW Sprint - Oct 28 - Nov 10, TSSW Sprint - Nov 11 - Nov 22 [ 966, 969 ]
            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, Tiago Ribeiro
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.