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

Please simplify the RPi ESS server API

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Epic Link:
    • Sprint:
      TSSW Sprint - Jan 03 - Jan 17
    • Team:
      Telescope and Site
    • Urgent?:
      No

      Description

      Please simplify the ESS RPi server's TCP/IP API to something like the following:

      • The server starts writing data after a client connects and issues the configure command.
      • The server stops writing data when a client disconnects.

      Thus the server would support just one command. Two subtleties:

      • The server should reject additional configure commands after the first one.
      • A new client should always have to send the configure command in order to receive data. The server should not remember configuration from an earlier client.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          rowen Russell Owen added a comment -

          Please also consider encoding the sensor name, timestamp, reply code, and sensor data fields as 4 separately named entities in published telemetry data, rather than as one heterogenous list of 4+ values. That would make the data and client code clearer.

          Show
          rowen Russell Owen added a comment - Please also consider encoding the sensor name, timestamp, reply code, and sensor data fields as 4 separately named entities in published telemetry data, rather than as one heterogenous list of 4+ values. That would make the data and client code clearer.
          Hide
          Permalink
          wvreeven Wouter van Reeven added a comment -

          ts_ess_common PR:

          https://github.com/lsst-ts/ts_ess_common/pull/12

          ts_ess_controller PR:

          https://github.com/lsst-ts/ts_ess_controller/pull/23

          ts_ess_csc PR:

          https://github.com/lsst-ts/ts_ess_csc/pull/58

           

          Show
          wvreeven Wouter van Reeven added a comment - ts_ess_common PR: https://github.com/lsst-ts/ts_ess_common/pull/12 ts_ess_controller PR: https://github.com/lsst-ts/ts_ess_controller/pull/23 ts_ess_csc PR: https://github.com/lsst-ts/ts_ess_csc/pull/58  
          Hide
          Permalink
          rowen Russell Owen added a comment -

          Reviewed on github

          Show
          rowen Russell Owen added a comment - Reviewed on github

            People

            Assignee:
            wvreeven Wouter van Reeven
            Reporter:
            rowen Russell Owen
            Reviewers:
            Russell Owen
            Watchers:
            Russell Owen, Wouter van Reeven
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.