-
Notifications
You must be signed in to change notification settings - Fork 33
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 this be used in ROS 2? #10
Comments
Good question. I haven't looked into ROS 2 control yet. Since the control algorithms are abstracted in a base class, these do not need to be touched. The ROS 1 integration is already a wrapper around them, so it might be comparably easy to make this ROS 2 compliant. If you have the time to look into it, you can take a look at the cartesian_controllers library that already did the ROS 2 port. |
any update on this one? |
I learned a bit more about ros2_control in the meantime and it should be relatively to port this package. If anyone is interested in doing this and to send in a pull request, that would be greatly appreciated. |
agreed, since the code is nicely structured, this shouldn't be too difficult. Might give this a go, but currently busy doing other things as well. For reference lbr-stack/lbr_fri_ros2_stack#163 |
Or what modifications do I need to make if I want to use it in ROS2
The text was updated successfully, but these errors were encountered: