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

Update the MTHexapod CSC to have the client in the CSC

    XMLWordPrintable

    Details

    • Story Points:
      3
    • Sprint:
      TSSW Sprint - Sep 27 - Oct 11
    • Team:
      Telescope and Site
    • Urgent?:
      No

      Description

      Once DM-31883 is implemented (or in progress) update the MTHexapod CSC so that the client is in the CSC and the server is in the mock low-level controller (the opposite of the current situation).

      In addition: adopt Tiago Ribeiro's suggestion to decouple the low-level controller's application state (which includes the usual CSC states) from the CSC's own summary state. When the CSC is enabled it should command the low-level controller into the enabled state. This decoupling should eliminate a source of nonstandard CSC behavior and makes the MTHexapod and MTRotator CSCs much more standard. It is also a good step towards allowing the low-level controllers to have simplified application states, should we decide to do that.

      This will require additional configuration for the CSC to specify the IP address of the low-level controller.

        Attachments

          Issue Links

            Activity

            Hide
            ttsai Te-Wei Tsai added a comment -

            Private IPs of hexapod PXIs on summit:
            1. camera hexapod: 10.9.57.226
            2. M2 hexapod: 10.9.57.231

            Show
            ttsai Te-Wei Tsai added a comment - Private IPs of hexapod PXIs on summit: 1. camera hexapod: 10.9.57.226 2. M2 hexapod: 10.9.57.231
            Show
            rowen Russell Owen added a comment - - edited Pull requests: https://github.com/lsst-ts/ts_hexrotcomm/pull/43 https://github.com/lsst-ts/ts_mthexapod/pull/53 https://github.com/lsst-ts/ts_mtrotator/pull/52 https://github.com/lsst-ts/ts_idl/pull/74 https://github.com/lsst-ts/ts_config_mttcs/pull/24 https://github.com/lsst-ts/ts_mthexapod/pull/55
            Hide
            ttsai Te-Wei Tsai added a comment -

            Reviewed in GitHub.

            Show
            ttsai Te-Wei Tsai added a comment - Reviewed in GitHub.
            Hide
            rowen Russell Owen added a comment -

            Thank you for a very helpful review. All changes merged to develop.

            I plan to hold off merging to master until we can test the code.

            We could tag ts_idl at any time, but I'd rather wait until we need it tagged, e.g. when we are closer to a new release cycle or ts_xml release.

            Show
            rowen Russell Owen added a comment - Thank you for a very helpful review. All changes merged to develop. I plan to hold off merging to master until we can test the code. We could tag ts_idl at any time, but I'd rather wait until we need it tagged, e.g. when we are closer to a new release cycle or ts_xml release.

              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.