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

Rewrite the Logic to Assign the DDS Telemetry Data

    XMLWordPrintable

    Details

      Description

      Rewrite the logic to assign the DDS telemetry data. This task will check the data to DDS is useful or not, and might add new field if needed. In addition, the DDS telemetry data should come from the GUI data directly.

      Some comment from Russell Owen in the slack channel to debug DM-30550:

      Te-Wei Tsai and I were talking. I suggested that the first step is for Te-Wei Tsai to print one or both of strutEncoderRaw and strutEncoder_microns (to stdout, a log, or a file – whatever is easy) from the dds telemetry packet when he publishes that packet. This will tell us if the problem is in the low-level controller or in the CSC. Also the code that sets these fields is suspicious and he will probably change it. But first I think it is important to prove whether the problem is actually in the low-level controller or in the CSC.

      A longer-term improvement we have tentatively agreed on is to make the telemetry sent to the CSC identical to the telemetry sent to the EUI. That way if we see a problem in one, we expect to see it in the other as well. Unfortunately the amount of data sent to the EUI is likely too much — it may overwhelm the python in the CSC. So a variant of that suggestion is to divide the telemetry into two pieces: one has all the “important” information that is sent to the CSC and the EUI. The other piece has additional information that is only sent to the EUI.
      This would be a big improvement over the present situation: the telemetry packet for the CSC is computed entirely separately from the telemetry packet sent to the EUI. A bug in one will not show up in the other.

        Attachments

        1. HexapodEUI.png
          HexapodEUI.png
          60 kB
        2. image (1).png
          image (1).png
          592 kB
        3. loggingDataInGUI.png
          loggingDataInGUI.png
          72 kB
        4. messages
          3.24 MB
        5. telemetry.txt
          2.09 MB

          Issue Links

            Activity

            ttsai Te-Wei Tsai created issue -
            ttsai Te-Wei Tsai made changes -
            Field Original Value New Value
            Link This issue is triggered by DM-30550 [ DM-30550 ]
            ttsai Te-Wei Tsai made changes -
            Description Rewrite the logic to assign the DDS telemetry data. This task will check the data to DDS is useful or not, and might add new field if needed. In addition, the DDS telemetry data should come from the GUI data directly. Rewrite the logic to assign the DDS telemetry data. This task will check the data to DDS is useful or not, and might add new field if needed. In addition, the DDS telemetry data should come from the GUI data directly.

            Some comment from [~rowen] in the slack channel to debug DM-30550:

            [~ttsai] and I were talking. I suggested that the first step is for [~ttsai] to print one or both of *strutEncoderRaw* and *strutEncoder_microns* (to stdout, a log, or a file -- whatever is easy) from the dds telemetry packet when he publishes that packet. This will tell us if the problem is in the low-level controller or in the CSC. Also the code that sets these fields is suspicious and he will probably change it. But first I think it is important to prove whether the problem is actually in the low-level controller or in the CSC.

            A longer-term improvement we have tentatively agreed on is to make the telemetry sent to the CSC identical to the telemetry sent to the EUI. That way if we see a problem in one, we expect to see it in the other as well. Unfortunately the amount of data sent to the EUI is likely too much — it may overwhelm the python in the CSC. So a variant of that suggestion is to divide the telemetry into two pieces: one has all the “important” information that is sent to the CSC and the EUI. The other piece has additional information that is only sent to the EUI.
            This would be a big improvement over the present situation: the telemetry packet for the CSC is computed entirely separately from the telemetry packet sent to the EUI. A bug in one will not show up in the other.
            hdrass Holger Drass made changes -
            Link This issue is blocked by SUMMIT-5251 [ SUMMIT-5251 ]
            hdrass Holger Drass made changes -
            Link This issue is blocked by SUMMIT-5251 [ SUMMIT-5251 ]
            hdrass Holger Drass made changes -
            Link This issue blocks SUMMIT-5251 [ SUMMIT-5251 ]
            ttsai Te-Wei Tsai made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            ttsai Te-Wei Tsai made changes -
            Story Points 2 1
            ttsai Te-Wei Tsai made changes -
            Attachment HexapodEUI.png [ 50305 ]
            ttsai Te-Wei Tsai made changes -
            Attachment loggingDataInGUI.png [ 50306 ]
            ttsai Te-Wei Tsai made changes -
            Story Points 1 2
            hdrass Holger Drass made changes -
            Attachment Screen Shot 2021-06-14 at 15.26.33.png [ 50307 ]
            hdrass Holger Drass made changes -
            Attachment image (1).png [ 50308 ]
            hdrass Holger Drass made changes -
            Attachment Screen Shot 2021-06-14 at 15.26.33.png [ 50307 ]
            ttsai Te-Wei Tsai made changes -
            Attachment messages [ 50400 ]
            Attachment telemetry.txt [ 50401 ]
            ttsai Te-Wei Tsai made changes -
            Reviewers Russell Owen [ rowen ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            rowen Russell Owen made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            ttsai Te-Wei Tsai made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            ttsai Te-Wei Tsai made changes -
            Link This issue is triggering DM-30811 [ DM-30811 ]
            ttsai Te-Wei Tsai made changes -
            Link This issue relates to DM-30814 [ DM-30814 ]
            hdrass Holger Drass made changes -
            Link This issue relates to SUMMIT-5251 [ SUMMIT-5251 ]
            hdrass Holger Drass made changes -
            Link This issue blocks SUMMIT-5251 [ SUMMIT-5251 ]
            hdrass Holger Drass made changes -
            Link This issue relates to SUMMIT-5404 [ SUMMIT-5404 ]
            hdrass Holger Drass made changes -
            Link This issue relates to SUMMIT-5796 [ SUMMIT-5796 ]

              People

              Assignee:
              ttsai Te-Wei Tsai
              Reporter:
              ttsai Te-Wei Tsai
              Reviewers:
              Russell Owen
              Watchers:
              Holger Drass, Russell Owen, Te-Wei Tsai
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Start date:
                End date:

                  Jenkins

                  No builds found.