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

ATDomeTrajectory sending negative dome position commands

    XMLWordPrintable

Details

    • Bug
    • Status: Done
    • Resolution: Done
    • None
    • None
    • None

    Description

      During the last run (see entry in log at 05:51am CLT) the ATDometrajectory CSC started sending negative positions to the ATDome CSC which only accepts a range of 0-360. Obviously the dome was in the wrong position but also no errors were thrown from either CSC which were apparent to the operator.

      https://confluence.lsstcorp.org/display/LSSTCOM/2021-01-21+Night+Log

      The issue was resolved by cycling both the Dome and ATDomeTrajectory CSCs. I can't remember off hand if we tried cycling just the ATDomeTrajectory or not...

      Attachments

        Issue Links

          Activity

            rowen Russell Owen added a comment -

            I proppose to fix this by allowing ATDome to accept azimuth outside the range 0-360 (and wrapping it to >= 0, < 360 for the low-level dome controller). It may be worth updating ATDomeTrajectory as well, but I consider that optional.

            rowen Russell Owen added a comment - I proppose to fix this by allowing ATDome to accept azimuth outside the range 0-360 (and wrapping it to >= 0, < 360 for the low-level dome controller). It may be worth updating ATDomeTrajectory as well, but I consider that optional.
            rowen Russell Owen added a comment - - edited

            I decided there was no point to updating ATDomeTrajectory. Making ATDome more generous in what azimuth angles it accepts is enough.

            In ts_xml I updated the description of the ATDome moveAzimuth command.
            In ts_ATDome I updated the code (a trivial change) and the unit test of moveAzimuth (a bigger change).

            Pull requests:

            rowen Russell Owen added a comment - - edited I decided there was no point to updating ATDomeTrajectory. Making ATDome more generous in what azimuth angles it accepts is enough. In ts_xml I updated the description of the ATDome moveAzimuth command. In ts_ATDome I updated the code (a trivial change) and the unit test of moveAzimuth (a bigger change). Pull requests: https://github.com/lsst-ts/ts_ATDome/pull/36 https://github.com/lsst-ts/ts_xml/pull/404

            LGTM

            wvreeven Wouter van Reeven added a comment - LGTM
            rowen Russell Owen added a comment -

            Released ts_ATDome v1.3.4

            rowen Russell Owen added a comment - Released ts_ATDome v1.3.4

            People

              rowen Russell Owen
              pingraham Patrick Ingraham
              Wouter van Reeven
              Patrick Ingraham, Russell Owen, Tiago Ribeiro, Wouter van Reeven
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.