Please Remove the Deprecated Telemetry Fields in Rotator CSC

XMLWordPrintable

Details

• Type: Story
• Status: Done
• Resolution: Done
• Fix Version/s: None
• Component/s:
• Labels:
• Epic Link:
• Team:
Telescope and Site
• Urgent?:
No

Description

Please remove the deprecated telemetry fields in rotator CSC with the new Tekniker rotator trajectory. The following fields are removed in the Simulink telemetry: ChAB_Cmd, ChA_Fb, and ChB_Fb.

Add the following telemetry fields:

  // Actuator motor current of actuator 1-2 (unit is A) double actMotorCurrent[2];   // Bus voltage of drive (unit is V) double busVoltage; 

Note. In DM-34177, the EUI will be used first to make sure the list of telemetry fields that needs to be updated. At this moment, this ticket is just a placeholder.

Activity

Hide
Russell Owen added a comment - - edited

Implement this as part of DM-31447. I looked at where this telemetry is used and found:

• ChAB_Cmd (called cmd_ch_ab in the CSC) is not used
• ChA_Fb and ChB_Fb (called ch_a_fb and ch_b_fb) are used to set the "calibrated" field of the motors telemetry topic. Set this to [NaN, Nan] in the new code and remeve the field in ts_xml 12 using DM-34311
Show
Russell Owen added a comment - - edited Implement this as part of DM-31447 . I looked at where this telemetry is used and found: ChAB_Cmd (called cmd_ch_ab in the CSC) is not used ChA_Fb and ChB_Fb (called ch_a_fb and ch_b_fb) are used to set the "calibrated" field of the motors telemetry topic. Set this to [NaN, Nan] in the new code and remeve the field in ts_xml 12 using DM-34311
Hide
Te-Wei Tsai added a comment -

I have tested the updated telemetry with the real hardware already. Please help to update the CSC based on this ticket description. Thanks!

Show
Te-Wei Tsai added a comment - I have tested the updated telemetry with the real hardware already. Please help to update the CSC based on this ticket description. Thanks!
Hide
Russell Owen added a comment -

I believe all work has been done for this in DM-34311 and DM-31447.

Show
Russell Owen added a comment - I believe all work has been done for this in DM-34311 and DM-31447 .

People

Assignee:
Russell Owen
Reporter:
Te-Wei Tsai
Watchers:
Russell Owen, Te-Wei Tsai
Votes:
0 Vote for this issue
Watchers:
2 Start watching this issue

Dates

Created:
Updated:
Resolved:

Jenkins

No builds found.