ArduPilot / ardupilot_gz

Tools for ArduPilot ROS2 integration and testing on ROS 2 humble
GNU General Public License v3.0
27 stars 19 forks source link

Copter: rename robot_state_publisher tf topic to /ap/tf #23

Closed pedro-fuoco closed 11 months ago

pedro-fuoco commented 11 months ago

I've explained in detail why robot_state_publisher's tf tree should be used when working with other ros applications in a Ardupilot Discourse post, as it follows ROS REP 105 tree structure.

Because of that, we should rename /ignore/tf | /ignore/tf_static to /ap/tf | ap/tf_static as this is the namespace currently used by AP_DDS_Client.

I will adequate the rviz launch calls in ardupilot_ros to match this namespace (by remapping the /tf topics). With this I will be able to get cartographer and navigation2 working correctly.

pedro-fuoco commented 11 months ago

@srmainwaring