-
Notifications
You must be signed in to change notification settings - Fork 387
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
What to do about ROS 2 support? #646
Comments
Hello @sea-bass ! I'm just getting to know ROS. I was wondering when a version of roslibjs and rosbridge_suite will be available in the rolling distribution that would allow this example action to be sent from roslibjs through rosbridge to the simulation. https://github.com/cyberbotics/webots_ros2/wiki/Example-Universal-Robots
Is there a timeline known or can you give an estimate? Thank you very much! |
You can use this PR branch right now, actually! |
@sea-bass |
The approach you described in that stackexchange question will install
|
@sea-bass That's good to know; I had no idea NPM could do that. I made the changes and installed ros2-actions with the first command. However, it didn't fix the problem, and I get the same errors on Rosbridge. So does that mean the problem is somewhere else than in roslib.js? |
Oh I just saw your stack exchange issue. I think you are using the ROS 1 compatible version of the action client, not the ROS 2. Try following the example here: https://github.com/RobotWebTools/roslibjs/blob/ros2-actions/examples/ros2_action_client.html |
Thanks, I got it working! I actually looked at the example, but I didn't notice the different classes for ros2. |
Yeah, the ROS 2 Implementation is totally different from the ROS 1, and this is admittedly very confusing. We're working on cutting a v2 version of this library, stripping away the ROS 1 support, and keeping the v1 as the "final" ROS 1 release. |
I tried to port "tf2_web_republisher" to ros2. I can get action for tf woking. I tested with action CLI. However, when I am using roslibjs I got this error
I build
When I use this parameter to call the action in CLI, It works just fine. Any suggestions? Thank you. |
Looking at your example, I think I think @pac48 is close to having a working prototype on both the tf2_web_republisher and roslibjs side. Just letting you know, as his fork has changed the message package to be more "correct" to how ROS 2 does things, with separate message packages: https://github.com/pac48/tf2_web_republisher |
Thank you for letting me know about @pac48 work. I think I will wait for him to finish the working prototype for both roslibjs and tf2_web_republisher. |
I think for now, the decision here is that roslibjs 2.0 will be adding some ROS 2 specific functionality but not regressing ROS 1. We should try our best to continue that strategy, but there may be another decision point later (hopefully after ROS 1 is officially EOL) |
I have recently implemented ROS 2 action support in
rosbridge_suite
in RobotWebTools/rosbridge_suite#886, and a working client inroslibjs
in #645.However, this leaves a few questions open regarding ROS 2 support for this library. Namely,
So, what should the plan be going forward?
develop
branch on ROS 1 and start tackling the CI/testing revamping on theros2
branch?ros2libjs
repo?The text was updated successfully, but these errors were encountered: