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

ROS namespaces #46

Open
adeguet1 opened this issue Dec 9, 2022 · 0 comments
Open

ROS namespaces #46

adeguet1 opened this issue Dec 9, 2022 · 0 comments
Assignees
@adeguet1

Comments

@adeguet1
Copy link
Collaborator

adeguet1 commented Dec 9, 2022

ROS node constructor can take an extra parameter to specify the namespace. We need to add this to the MRMLROS2NodeNode to support multiple robots. We can assume that we will use one node node per robot.
@adeguet1 adeguet1 self-assigned this Dec 9, 2022
adeguet1 added a commit that referenced this issue Jan 4, 2023
@adeguet1
* use try/catch around rclcpp::init to initialize only once #45
5f5d54f 
* removed old code so logic only uses new MRML nodes #32
* ROS nodes now added to SlicerROS2 logic when restored from xml #14
* `CreateAndAddXXX` methods moved from logic to ROS node since the publishers/subscribers... are attached to the node #46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adeguet1 adeguet1

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@adeguet1