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

Update ts_ScriptQueue to use the dds version of ts_salobj

    XMLWordPrintable

    Details

      Description

      Update ts_ScriptQueue to use the version of ts_salobj based on dds. This is primarily intended to exercise the new version of ts_salobj.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment - - edited

            Additional changes:

            • Use a single salobj.Remote in the queue model to communicate with all scripts. This is faster and more efficient than building a separate remote to talk to each script. It does require a bit of care to set the ScriptID field (which is the name of the SAL index field for Script topics) before sending a command to a script.
            • Add a log attribute to QueueModel and ScriptInfo.

            Pull request: https://github.com/lsst-ts/ts_scriptqueue/pull/27

            Show
            rowen Russell Owen added a comment - - edited Additional changes: Use a single salobj.Remote in the queue model to communicate with all scripts. This is faster and more efficient than building a separate remote to talk to each script. It does require a bit of care to set the ScriptID field (which is the name of the SAL index field for Script topics) before sending a command to a script. Add a log attribute to QueueModel and ScriptInfo. Pull request: https://github.com/lsst-ts/ts_scriptqueue/pull/27
            Hide
            ecoughlin Eric Coughlin added a comment -

            Approved on github.

            Show
            ecoughlin Eric Coughlin added a comment - Approved on github.
            Hide
            rowen Russell Owen added a comment -

            Merged to develop and tagged as v2.0b1; this tag also includes the conversion to use dds salobj. I plan to hold off merging to master until dds salobj can communication with SAL 3.9.

            Show
            rowen Russell Owen added a comment - Merged to develop and tagged as v2.0b1; this tag also includes the conversion to use dds salobj. I plan to hold off merging to master until dds salobj can communication with SAL 3.9.

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Reviewers:
              Eric Coughlin
              Watchers:
              Eric Coughlin, Russell Owen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: