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

Update ts_ATDome for changes to vendor code

    Details

      Description

      The vendor updated the low level ATDome code and the CSC can no longer parse full status. The error is:

      Status request failed: Cound not parse 'Top Comm Link OK: 1' as SCB radio link OK: +(\d)
      

      Tiago Ribeiro kindly got me an example of the new full status output:

      MAIN AJAR 000
      DROP AJAR 000
      [OFF] 00
      POSN 22.90
      RR 000
      Emergency Stop Active: 0
      Top Comm Link OK: 0
      Home Azimuth:  0.00
      High Speed (degrees): 5.00
      Coast (degrees): 0.50
      Tolerance (degrees): 1.00
      Encoder Counts per 360: 4018143232
      Encoder Counts: 81630126151
      Last Azimuth GoTo: 20.00
      Azimuth Move Timeout (secs): 120
      Rain-Snow enabled: 0
      Cloud Sensor enabled: 1
      Watchdog Reset Time: 600
      Dropout Timer: 5
      Reverse Delay: 5
      Main Door Encoder Closed: 0
      Main Door Encoder Opened: 0
      Dropout Encoder Closed: 0
      Dropout Encoder Opened: 0
      Door Move Timeout (secs): 10
      

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment -

            The only change seems to be "SCB radio link" to "Top Comm Link"

            Show
            rowen Russell Owen added a comment - The only change seems to be "SCB radio link" to "Top Comm Link"
            Hide
            rowen Russell Owen added a comment - - edited

            Pull request: https://github.com/lsst-ts/ts_ATDome/pull/17

            In addition Tiago wanted the same fix applied to v0.8.0.
            That pull request: https://github.com/lsst-ts/ts_ATDome/pull/18
            (note that I had to make a new branch `0.8.0` to make this pull request)

            Show
            rowen Russell Owen added a comment - - edited Pull request: https://github.com/lsst-ts/ts_ATDome/pull/17 In addition Tiago wanted the same fix applied to v0.8.0. That pull request: https://github.com/lsst-ts/ts_ATDome/pull/18 (note that I had to make a new branch `0.8.0` to make this pull request)
            Hide
            rowen Russell Owen added a comment -

            Tiago reviewed the two pull requests on github and approved both.

            Show
            rowen Russell Owen added a comment - Tiago reviewed the two pull requests on github and approved both.
            Hide
            rowen Russell Owen added a comment -

            Merged the main pull request to develop and master and tagged as v1.0.0

            Merged the other pull request to 0.8.0 and tagged as v0.8.0b.

            I will upload docs once I get a working LSST stack again.

            Show
            rowen Russell Owen added a comment - Merged the main pull request to develop and master and tagged as v1.0.0 Merged the other pull request to 0.8.0 and tagged as v0.8.0b. I will upload docs once I get a working LSST stack again.

              People

              • Assignee:
                rowen Russell Owen
                Reporter:
                rowen Russell Owen
                Reviewers:
                Tiago Ribeiro
                Watchers:
                Russell Owen, Tiago Ribeiro
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: