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.

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

Add a stop_all_motors service #30

Open
mklomp opened this issue Oct 18, 2023 · 3 comments
Open

Add a stop_all_motors service #30

mklomp opened this issue Oct 18, 2023 · 3 comments
Milestone
0.2

Comments

@mklomp
Copy link
Contributor

mklomp commented Oct 18, 2023

While developing, it is quite useful to stop all motors with one command (and optionally, also disable the servo's). This service can then also be used in the python API and web interface to stop all motors from spinning.
@mklomp mklomp added this to the 0.2 milestone Oct 18, 2023
@ArendJan
Copy link
Contributor

That would be nice indeed!

Would this be a 'stop any moving motor right now' service or a 'disable all motors until enable again' service?
Or even both?

@mklomp
Copy link
Contributor Author

mklomp commented Oct 23, 2023

My first reaction would be to have a 'stop all motors right now' behaviour. I think otherwise we potentially introduce 'bugs' where motors are not responding while they are just disabled.

@ArendJan
Copy link
Contributor

After discussion:

  • add in telemetrix node a service if there is any motor
  • on service call, send motorspeed =0% to all motors

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
0.2
Development

No branches or pull requests
2 participants
@mklomp @ArendJan