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.

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

ros2 node list doesn't include node namespace #105

Closed
dhood opened this issue Jun 21, 2018 · 3 comments · Fixed by #146
Closed

ros2 node list doesn't include node namespace #105

dhood opened this issue Jun 21, 2018 · 3 comments · Fixed by #146
Assignees
Labels
enhancement New feature or request

Comments

@dhood
Copy link
Member

dhood commented Jun 21, 2018

Requires ros2/rmw#142

I first noticed this running a node with a remapped namespace, but it also happens if I hard-code a namespace. It happens:

  • hard-coded or using static remapping
  • if the node uses rclcpp or rclpy
  • for a daemon using connext, fastrtps, or opensplice.

E.g.:

$ ros2 run demo_nodes_cpp talker  # (modified to use hard-coded namespace)
[INFO] [my_ns.talker]: Publishing: 'Hello World: 1'

$ ros2 node list
talker

I would consider this a bug because I expected the output to include the node's namespace, unless we are aware that this is not supported?

@dhood dhood added the bug Something isn't working label Jun 21, 2018
@sloretz
Copy link
Contributor

sloretz commented Jun 21, 2018

just noting ROS 1 includes the namespace

$ rosnode list
/gazebo
/gazebo_gui
/iris/joy_node
/iris/robot_state_publisher
/iris/roll_pitch_yawrate_thrust_controller_node
/iris/rotors_joy_interface
/rosout

@dhood
Copy link
Member Author

dhood commented Jun 21, 2018

I think this has just never been supported. In Ardent it also didn't expand. I will re-classify this as an enhancement.

@dhood dhood added enhancement New feature or request and removed bug Something isn't working labels Jun 21, 2018
@dirk-thomas
Copy link
Member

The underlying API doesn't provide the namespace so the command line tools can't show them atm. This needs to be addressed in ros2/rmw#142 first.

@dirk-thomas dirk-thomas assigned mjcarroll and unassigned tfoote Aug 23, 2018
@mjcarroll mjcarroll added the in progress Actively being worked on (Kanban column) label Aug 29, 2018
@mjcarroll mjcarroll added in review Waiting for review (Kanban column) and removed in progress Actively being worked on (Kanban column) labels Aug 29, 2018
@dirk-thomas dirk-thomas self-assigned this Sep 6, 2018
@dirk-thomas dirk-thomas removed the in review Waiting for review (Kanban column) label Sep 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants