Merge pull request #30 from christianrauch/resolve_topic

resolve topic name before applying the image transport suffix
christianrauch · Mar 21, 2024 · 6ec122f · 6ec122f
2 parents 21d1849 + bc71089
commit 6ec122f
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/AprilTagNode.cpp b/src/AprilTagNode.cpp
@@ -98,7 +98,12 @@ AprilTagNode::AprilTagNode(const rclcpp::NodeOptions& options)
     cb_parameter(add_on_set_parameters_callback(std::bind(&AprilTagNode::onParameter, this, std::placeholders::_1))),
     td(apriltag_detector_create()),
     // topics
-    sub_cam(image_transport::create_camera_subscription(this, "image_rect", std::bind(&AprilTagNode::onCamera, this, std::placeholders::_1, std::placeholders::_2), declare_parameter("image_transport", "raw", descr({}, true)), rmw_qos_profile_sensor_data)),
+    sub_cam(image_transport::create_camera_subscription(
+        this,
+        this->get_node_topics_interface()->resolve_topic_name("image_rect"),
+        std::bind(&AprilTagNode::onCamera, this, std::placeholders::_1, std::placeholders::_2),
+        declare_parameter("image_transport", "raw", descr({}, true)),
+        rmw_qos_profile_sensor_data)),
     pub_detections(create_publisher<apriltag_msgs::msg::AprilTagDetectionArray>("detections", rclcpp::QoS(1))),
     tf_broadcaster(this)
 {