Skip to content
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.

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

Can dbc updates #741

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Can dbc updates #741

wants to merge 3 commits into from

Conversation

ss32
Copy link

@ss32 ss32 commented Feb 10, 2024

Updates to scripts related to the CAN DBC. Currently the DBC creation fails due to invalid class constructors and can_dbc_example.py fails due to improper datatype comparisons and invalid message names. The CAN DBC example script also ends in an infinite loop with no means to cleanly exit and stop the ODRive, so I've added a path to allow for a graceful shutdown of the code and ODrive.

Scott added 3 commits February 10, 2024 06:41
Modifies calls to can.Signal constructors to use NamedSignalConversion
class.
 * Corrects bugs in current_state to check against string instead of hex. Retval is no longer hex
 * Adds rate limiting to velocity printout so the terminal isn't spammed with values
 * Allows for a graceful shutdown of the closed loop control portion
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Scott seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

None yet

2 participants