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

[Backport rolling] Add ROS namespaces to GZ topics #517

Merged
merged 3 commits into from
Apr 1, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
40 changes: 37 additions & 3 deletions ros_gz_bridge/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ Now we start the ROS listener.

```
# Shell B:
. /opt/ros/galactic/setup.bash
. /opt/ros/rolling/setup.bash
ros2 topic echo /chatter
```

Expand Down Expand Up @@ -134,7 +134,7 @@ Now we start the ROS talker.

```
# Shell C:
. /opt/ros/galactic/setup.bash
. /opt/ros/rolling/setup.bash
ros2 topic pub /chatter std_msgs/msg/String "data: 'Hi'" --once
```

Expand Down Expand Up @@ -172,7 +172,7 @@ Now we start the ROS GUI:

```
# Shell C:
. /opt/ros/galactic/setup.bash
. /opt/ros/rolling/setup.bash
ros2 run rqt_image_view rqt_image_view /rgbd_camera/image
```

Expand Down Expand Up @@ -290,9 +290,43 @@ To run the bridge node with the above configuration:
ros2 run ros_gz_bridge parameter_bridge --ros-args -p config_file:=$WORKSPACE/ros_gz/ros_gz_bridge/test/config/full.yaml
```

## Example 6: Using ROS namespace with the Bridge

When spawning multiple robots inside the same ROS environment, it is convenient to use namespaces to avoid overlapping topic names.
There are three main types of namespaces: relative, global (`/`) and private (`~/`). For more information, refer to ROS documentation.
Namespaces are applied to Gazebo topic both when specified as `topic_name` as well as `gz_topic_name`.

By default, the Bridge will not apply ROS namespace on the Gazebo topics. To enable this feature, use parameter `expand_gz_topic_names`.
Let's test our topic with namespace:

```bash
# Shell A:
. ~/bridge_ws/install/setup.bash
ros2 run ros_gz_bridge parameter_bridge chatter@std_msgs/msg/[email protected] \
--ros-args -p expand_gz_topic_names:=true -r __ns:=/demo
```

Now we start the Gazebo Transport listener.

```bash
# Shell B:
gz topic -e -t /demo/chatter
```

Now we start the ROS talker.

```bash
# Shell C:
. /opt/ros/rolling/setup.bash
ros2 topic pub /demo/chatter std_msgs/msg/String "data: 'Hi from inside of a namespace'" --once
```

By changing `chatter` to `/chatter` or `~/chatter` you can obtain different results.

## API

ROS 2 Parameters:

* `subscription_heartbeat` - Period at which the node checks for new subscribers for lazy bridges.
* `config_file` - YAML file to be loaded as the bridge configuration
* `expand_gz_topic_names` - Enable or disable ROS namespace applied on GZ topics.
14 changes: 12 additions & 2 deletions ros_gz_bridge/src/ros_gz_bridge.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@
#include "bridge_handle_ros_to_gz.hpp"
#include "bridge_handle_gz_to_ros.hpp"

#include <rclcpp/expand_topic_or_service_name.hpp>

namespace ros_gz_bridge
{

Expand All @@ -30,6 +32,7 @@ RosGzBridge::RosGzBridge(const rclcpp::NodeOptions & options)

this->declare_parameter<int>("subscription_heartbeat", 1000);
this->declare_parameter<std::string>("config_file", "");
this->declare_parameter<bool>("expand_gz_topic_names", false);

int heartbeat;
this->get_parameter("subscription_heartbeat", heartbeat);
Expand All @@ -43,14 +46,21 @@ void RosGzBridge::spin()
if (handles_.empty()) {
std::string config_file;
this->get_parameter("config_file", config_file);
bool expand_names;
this->get_parameter("expand_gz_topic_names", expand_names);
const std::string ros_ns = this->get_namespace();
const std::string ros_node_name = this->get_name();
if (!config_file.empty()) {
auto entries = readFromYamlFile(config_file);
for (const auto & entry : entries) {
for (auto & entry : entries) {
if (expand_names) {
entry.gz_topic_name = rclcpp::expand_topic_or_service_name(
entry.gz_topic_name, ros_node_name, ros_ns, false);
}
this->add_bridge(entry);
}
}
}

for (auto & bridge : handles_) {
bridge->Spin();
}
Expand Down
Loading