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

Add units to M1M3 telemetry, degrees as units instead of radians

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ts_main_telescope
    • Labels:
      None

      Description

      M1M3 SS telemetry (ts_xml/sal_interfaces/MTM1M3/MTM1M3_Telemetry.xml) is missing units (A for current,..). It will be nice to include those as implemented in cRIO (e.g. A for current is measured as voltage on cRIO module, where there is a conversion between voltage to current). Also lot of angles (xRotation,..) claims either uniteless or deg, but in fact the unit as implemented is radians. That needs to be checked and changed.

        Attachments

          Issue Links

            Activity

            Hide
            pkubanek Petr Kubanek added a comment -

            Please see PR. PR for GUI is in progress, will follow soon.

            Show
            pkubanek Petr Kubanek added a comment - Please see PR. PR for GUI is in progress, will follow soon.
            Hide
            rbovill Rob Bovill added a comment -

            https://github.com/lsst-ts/ts_xml/pull/417

             

            Units have been updated.

            Show
            rbovill Rob Bovill added a comment - https://github.com/lsst-ts/ts_xml/pull/417   Units have been updated.

              People

              Assignee:
              pkubanek Petr Kubanek
              Reporter:
              pkubanek Petr Kubanek
              Reviewers:
              Bo Xin [X] (Inactive), Dave Mills, Felipe Daruich, Rob Bovill, Te-Wei Tsai
              Watchers:
              Andy Clements, Bo Xin [X] (Inactive), Dave Mills, Doug Neill, Felipe Daruich, Petr Kubanek, Rob Bovill, Sandrine Thomas, Te-Wei Tsai
              Votes:
              0 Vote for this issue
              Watchers:
              9 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Start date:

                  Jenkins

                  No builds found.