The nodehandle namespace of send_topics and recv_topics should be public
shupx opened this issue · comments
In ros_topics.yaml, you can specify the send_topics and recv_topics names in either an absolute or a relative way. However, the topic names will have a ros node name prefix if they are specified in a relative way. For example, - topic_name: chatter
will result in the subscribe of /swarm_bridge_node/chatter
topic.
This is because that the nodehandles of advertising and subscribe are public rather than private:
swarm_ros_bridge/src/bridge_node.cpp
Line 180 in 4bc8da2
swarm_ros_bridge/src/bridge_node.cpp
Line 293 in 4bc8da2
swarm_ros_bridge/src/bridge_node.cpp
Line 302 in 4bc8da2
This is unwanted and we need the prefix only be defined by <group ns=''>
tag in the launch file.
So the nodehandles should be public.
This is fixed in commit c31fc94. Now relative send_topic and recv_topic names are also supported.