moveit / moveit2

:robot: MoveIt for ROS 2
https://moveit.ai/
BSD 3-Clause "New" or "Revised" License
1.05k stars 513 forks source link

Remove intraprocess comm warning #2752

Closed SammyRamone closed 5 months ago

SammyRamone commented 6 months ago

Description

This PR fixes https://github.com/ros-planning/moveit2/issues/2632. It simply removes the warning altogether. Currently, the intraprocess communication still lacks the necessary support for the KEEP_ALL QoS (see https://github.com/ros2/rclcpp/issues/752). As far as I understand, as long as this issue is not solved, Moveit does not work with intraprocess communication. I opened a new issue (https://github.com/ros-planning/moveit2/issues/2753) for reintroducing this warning as soon as the intraprocess communication is usable, so that this feature does not get forgotten.

Checklist