Skip to content

Support for other cyclic messages #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
BrechtSerckx opened this issue Mar 20, 2025 · 2 comments
Closed

Support for other cyclic messages #13

BrechtSerckx opened this issue Mar 20, 2025 · 2 comments

Comments

@BrechtSerckx
Copy link
Contributor

Hi!

There are 7 configurable cyclic messages:

  • Heartbeat
  • Get_Encoder_Estimates
  • Get_Iq
  • Get_Error
  • Get_Temperature
  • Get_Bus_Voltage_Current
  • Get_Torques

However, the ODriveArduino library only has support for callback handlers of 3:

  • Heartbeat
  • Get_Encoder_Estimates
  • Get_Torques

Are there any plans to support them all?

For my project, I will need support for Get_Bus_Voltage_Current and maybe
Get_Temperature. I can implement similar callback handlers as the existing ones for these messages, and also for the other cyclic messages if desired.

Would you have interest in merging that?
@madcowswe
Copy link
Member

Hi,
If you are able to implement them in a similar style and organization as the existing ones then I expect we'd be interesting in merging that yes. Thank you!

This was referenced Apr 23, 2025
Merged
Merged
@BrechtSerckx
Copy link
Contributor Author

Implemented in #15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@madcowswe @BrechtSerckx