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

Remove all use of SALPY from ts_salobj

    XMLWordPrintable

    Details

      Description

      Once Rob Bovill has the integration test for [C++, java, Python] <-> [C++, java, Python] SAL communication working, remove all use of SALPY from ts_salobj.

      Specifically:

      • Remove the two test_x_to_either.py tests and the two associated minimal controllers.
      • Remove ts_sal as a dependency in the ups table
      • Update the documentation to remove all reference to SALPY.
      • Update Jenkinsfile to remove commands:/pytest --ignore tests/test_salpy_to_either.py ...
      • Update conda/meta.yaml to remove make_salpy_libs.py Test

      Do a global search for SALPY and "_to_either" for any other things to delete.

        Attachments

          Activity

          Hide
          rowen Russell Owen added a comment -
          Show
          rowen Russell Owen added a comment - Pull request: https://github.com/lsst-ts/ts_salobj/pull/211
          Hide
          wvreeven Wouter van Reeven added a comment -

          Reviewed in GitHub.

          Show
          wvreeven Wouter van Reeven added a comment - Reviewed in GitHub.
          Hide
          rowen Russell Owen added a comment -

          Merged to develop. I'll release this version or the previous one soon, depending on when the C++ and Java inter-language SAL communication test runners are accepted.

          Show
          rowen Russell Owen added a comment - Merged to develop. I'll release this version or the previous one soon, depending on when the C++ and Java inter-language SAL communication test runners are accepted.

            People

            Assignee:
            rowen Russell Owen
            Reporter:
            rowen Russell Owen
            Reviewers:
            Wouter van Reeven
            Watchers:
            Dave Mills, Rob Bovill, Russell Owen, Tiago Ribeiro, Wouter van Reeven
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.