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

Merge ATThermoelectricCooler CSC into ATWhiteLightSource

    XMLWordPrintable

    Details

      Description

      Chiller and White Light CSCs were separated at some point in the past because they were going to be in different languages and written by different people. Now they're both python and I'm working on both. So we've decided to simplify them a single CSC rather than a pair of CSCs with a third CSC coordinating.

      This task is to bring the currently-existing code for the chiller into the White Light CSC, merge their SAL topics, and get a docker environment up and running for the new unified CSC.

        Attachments

          Activity

          Show
          cwinslow Colin Winslow [X] (Inactive) added a comment - merged code,  https://github.com/lsst-ts/ts_ATWhiteLightSource/commit/863d350474420e60bbebe79374766347c2bd4bc1 merged sal commands,  https://github.com/lsst-ts/ts_xml/commit/d41ec9d815a18fb97d57dcead15347218d1b1626
          Hide
          cwinslow Colin Winslow [X] (Inactive) added a comment -

          also spent some time resolving an issue that was preventing the white light CSC from working with the latest salobj in the develop-env docker container. 

          Show
          cwinslow Colin Winslow [X] (Inactive) added a comment - also spent some time resolving an issue that was preventing the white light CSC from working with the latest salobj in the develop-env docker container. 
          Hide
          cwinslow Colin Winslow [X] (Inactive) added a comment -

          This is another one that I finished a while ago. The chiller communication class I showed you yesterday was all already part of ATWhiteLight. Here's a link to the ATWhiteLight repo with the chiller stuff merged in: https://github.com/lsst-ts/ts_ATWhiteLightSource/tree/develop/python/lsst/ts/ATWhiteLightSource

          The XML part of the merge is here: https://github.com/lsst-ts/ts_xml/pull/85

          Rob has already run his tests on this branch so I assigned him the PR review.

          Show
          cwinslow Colin Winslow [X] (Inactive) added a comment - This is another one that I finished a while ago. The chiller communication class I showed you yesterday was all already part of ATWhiteLight. Here's a link to the ATWhiteLight repo with the chiller stuff merged in:  https://github.com/lsst-ts/ts_ATWhiteLightSource/tree/develop/python/lsst/ts/ATWhiteLightSource The XML part of the merge is here:  https://github.com/lsst-ts/ts_xml/pull/85 Rob has already run his tests on this branch so I assigned him the PR review.
          Hide
          jbuffill James Buffill [X] (Inactive) added a comment -

          Merge in (added) following files:

          • chillerComponent.py
          • chillerEncoder.py
          • chillerModel.py
          • chillerResponseProcessor.py
          • sockscript.py 
          Show
          jbuffill James Buffill [X] (Inactive) added a comment - Merge in (added) following files: chillerComponent.py chillerEncoder.py chillerModel.py chillerResponseProcessor.py sockscript.py 
          Hide
          jbuffill James Buffill [X] (Inactive) added a comment -

          Good to go!

          Show
          jbuffill James Buffill [X] (Inactive) added a comment - Good to go!
          Hide
          jbuffill James Buffill [X] (Inactive) added a comment -

          Done

          Show
          jbuffill James Buffill [X] (Inactive) added a comment - Done

            People

            Assignee:
            cwinslow Colin Winslow [X] (Inactive)
            Reporter:
            cwinslow Colin Winslow [X] (Inactive)
            Reviewers:
            James Buffill [X] (Inactive)
            Watchers:
            Colin Winslow [X] (Inactive), James Buffill [X] (Inactive), Patrick Ingraham
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.