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

Add current limiting on motors #23

Open
jkleiber opened this issue Jan 15, 2023 · 2 comments
Open

Add current limiting on motors #23

jkleiber opened this issue Jan 15, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@jkleiber
Copy link
Contributor

Summary
Motors should have current limits enforced in software to prevent damage to our electronics. This should be worked after #22 adds the current limits to the parameters TOML file.

Work

  • Add current limiting to motors

Verification

  • CI passes
  • Current limits are applied to the motors
@jkleiber jkleiber added the enhancement New feature or request label Jan 15, 2023
@jkleiber jkleiber added this to the Base Actions milestone Jan 15, 2023
@danielbrownmsm
Copy link
Collaborator

note: this is absolutely necessary for the neo550s, as they smoke easily without it
less so the other motors, so really we just need to throw this at the claw

@jkleiber
Copy link
Contributor Author

@AbbeySieg This is related to #22 , and you can work both of these at the same time on a single new branch. Assigning this to you for now

@danielbrownmsm danielbrownmsm added long-term stuff that we don't necessarily have to do right now and removed long-term stuff that we don't necessarily have to do right now labels Apr 4, 2023
@danielbrownmsm danielbrownmsm self-assigned this Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants