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

MTRotator: use tv_sec instead of mjd when reading time from the low-level controller

    XMLWordPrintable

    Details

      Description

      The low-level controller supplies time in two different ways in telemetry and config headers: mjd/frac_mjd and tv_sec/tv_nsec. It turns out the former values are not MJD at all (they are just restatements of tv_sec and tv_nsec). Use the vt_sec/tv_nsec values instead.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment - - edited

            I also made a trivial change to salobj: the CSC commander now rounds values to 2 digits instead of 4, to make telemetry less noisy.

            Pull requests:

            Show
            rowen Russell Owen added a comment - - edited I also made a trivial change to salobj: the CSC commander now rounds values to 2 digits instead of 4, to make telemetry less noisy. Pull requests: https://github.com/lsst-ts/ts_mtrotator/pull/31 https://github.com/lsst-ts/ts_salobj/pull/171
            Hide
            rowen Russell Owen added a comment -

            Reviewed on github

            Show
            rowen Russell Owen added a comment - Reviewed on github
            Hide
            rowen Russell Owen added a comment -

            Released ts_salobj v6.2.0 (a minor change to v6.2.0.beta1) and ts_mtrotator v0.10.2 (since it is a bug fix release).

            Show
            rowen Russell Owen added a comment - Released ts_salobj v6.2.0 (a minor change to v6.2.0.beta1) and ts_mtrotator v0.10.2 (since it is a bug fix release).

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Reviewers:
              Tiago Ribeiro
              Watchers:
              Bo Xin [X] (Inactive), Russell Owen, Te-Wei Tsai, Tiago Ribeiro
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.