-
Notifications
You must be signed in to change notification settings - Fork 525
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
diff drive skid steer #138
diff drive skid steer #138
Conversation
A visual inspection of the code looks OK. There are unit tests (horray!) and the controller has been validated in different hardware platforms, so from my part: +1 It's great to see that writing a new controller was not necessary in the end. Good stuff guys! |
I'll fix this build tonight. |
Looks great, thanks very much for the work on this. As noted in the sig mailing list, I am having a weird issue with my system using this controller. However, I suspect it's a problem with my hardware node, or elsewhere in my system, rather than with the controller specifically. +1 from me. |
Test xacros fixed. |
This PR is an update from the hydro-devel branch of ros_controllers from PAL Robotics, and it depends on #137.
It adds support for multiple wheels per side to the diff_drive_controller.
This version of the diff_drive_controller is the one that is currently running on all PAL Robotics platforms and on the Jackal from Clearpath Robotics.
Related PRs:
pal-robotics-forks#42
#10