2023-02-03 - ZP-CV Communication Units and Ranges

Owned by R D, created
2023-02-03
1 min read

Update the title with the “name of change' after the dash

Change Description

ZeroPilot, IMACS, CV Airside

ZP-CV Communication Units and Ranges

@R D

Discord

Description of Change

Add units (preferably in SI) to .

Implementation

Ensure all stakeholders agree on the units and expected ranges used, adding them to documentation.

Reason for Change

Avoid a https://en.wikipedia.org/wiki/Mars_Climate_Orbiter incident where the interface sends values in one unit while another interface receives them in another unit causing error.

Priority

Medium

Impact of Not Responding to Change

Unclear what units are being communicated, potentially causing incorrect movement commands and incorrect data telemetry reporting.

What Groups, People, Sub-teams Need to be Notified?

  • Firmware EFS: @Aaditya Chaudhary

  • CV: @Aydan Jiwani (Deactivated) @Amy Hu @R D

Change Impact

Additional Parts/Resources Required and Costs

Time spent discussing and writing documentation.

Impact on deadlines

Little impact as units are implicitly known by stakeholders.

Alternatives and Recommendations

Do-nothing option, as @Aaditya Chaudhary has written the driver on both sides and so everyone can ask him instead.

Comments

N/A

Make sure to create a thread in the ‘rfc’ chat in discord (here). Share a link to your rfc in your post so it can be approved!

Change Request Sign Off <System Integration>

Status

Accepted / On Hold / Rejected / More info Requested

Comments

Signatures/Name