Skip to content

Update innotronic creation#40

Open
JensOgorek wants to merge 2 commits intomainfrom
update_innotronic_creation
Open

Update innotronic creation#40
JensOgorek wants to merge 2 commits intomainfrom
update_innotronic_creation

Conversation

@JensOgorek
Copy link
Copy Markdown

Motivation

Adapt to the updated Lizard InnotronicDriveMotor module. In addition, motor currents are now included in the core output so that current monitoring and debugging are possible at the application level.

Implementation

  • InnotronicTracksHardware updated to the new Lizard module signature (construction via InnotronicDriveMotor(can, node_id) and InnotronicWheels).
  • core_message_fields extended with left.current_m1, left.current_m2, right.current_m1, right.current_m2 (4 values, 2 per side — the Innotronic driver reports the current of motor 1 and motor 2 per module in amperes).
  • handle_core_output stores the four current values in left_current_m1, left_current_m2, right_current_m1, right_current_m2.

Note / Follow-ups

If further values from the Lizard module are needed (e.g. status flags, temperatures, position ticks), they have to be added analogously in core_message_fields and handle_core_output. For now only the currents are exposed — sufficient for initial logging/debugging; more can be added on demand.

Progress

  • I chose a meaningful title that completes the sentence: "If applied, this PR will..."
  • I chose meaningful labels (if GitHub allows me to so).
  • I documented breaking changes and set the 'breaking change' label if needed
  • The implementation is complete.
  • Tests with a real hardware have been successful (or are not necessary).
  • Pytests have been added (or are not necessary).
  • Documentation has been added (or is not necessary).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant