ros-visualization / rqt_graph

http://wiki.ros.org/rqt_graph
26 stars 27 forks source link

Namespaces include the node name where they previously appear to not do so #6

Closed wjwwood closed 7 years ago

wjwwood commented 7 years ago

While reviewing this tutorial for ROS Lunar:

http://wiki.ros.org/ROS/Tutorials/UnderstandingTopics

I noticed that the current rqt_graph rendering looks different than the historical screenshots in the wiki page. For example:

screenshot from 2017-05-11 15-20-38

If I uncheck the namespaces option it looks like the screenshot currently on the wiki:

screenshot from 2017-05-11 15-21-26

However, I feel that it should not require me doing that. That's why I think that the namespace is incorrectly including the node name at all times. Based on the code and rosnode info I do not believe these nodes are actually in namespaces:

% rosnode info /turtlesim 
--------------------------------------------------------------------------------
Node [/turtlesim]
Publications: 
 * /turtle1/color_sensor [turtlesim/Color]
 * /rosout [rosgraph_msgs/Log]
 * /turtle1/pose [turtlesim/Pose]

Subscriptions: 
 * /turtle1/cmd_vel [geometry_msgs/Twist]

Services: 
 * /clear
 * /turtle1/teleport_relative
 * /turtlesim/set_logger_level
 * /spawn
 * /reset
 * /kill
 * /turtlesim/get_loggers
 * /turtle1/teleport_absolute
 * /turtle1/set_pen

contacting node http://injera:37136/ ...
Pid: 28751
Connections:
 * topic: /rosout
    * to: /rosout
    * direction: outbound
    * transport: TCPROS
 * topic: /turtle1/cmd_vel
    * to: /teleop_turtle (http://injera:46066/)
    * direction: inbound
    * transport: TCPROS
dirk-thomas commented 7 years ago

Addressed in #8.

dirk-thomas commented 7 years ago

Another patch to get rid of the unintentional color item: ros-visualization/qt_gui_core#86