-
Notifications
You must be signed in to change notification settings - Fork 498
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
kinematics_plugin_loader port to ROS 2 #74
kinematics_plugin_loader port to ROS 2 #74
Conversation
Same as #62 (comment), consider the changes please. |
|
||
// ros::NodeHandle nh("~"); | ||
//TODO (anasarrak): Add the appropriate node name for ROS2 | ||
auto node = rclcpp::Node::make_shared("talker"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a big TODO. Let's do this the right way now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mlautman any suggestion?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the approach that was used for the controller manager where the node is passed in during initialization as it seems most flexible.
Closed in favor of #107 |
* Fix headers * Port source and launch files of planning scene ROS API * Fix prompts and add xterm to launch * Update documentation * Add info about xterm * Remove debugging info and add todo * Update doc/planning_scene_ros_api/planning_scene_ros_api_tutorial.rst * Remove unnecessary note. * Fix prompts Co-authored-by: Marq Rasmussen <[email protected]> Co-authored-by: Jafar Abdi <[email protected]>
No description provided.