search

moveit / moveit2

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

Reintroduce Intraprocess Communications to Servo when Supported by rclcpp #2753

Closed SammyRamone closed 4 weeks ago

SammyRamone commented 4 months ago

Is your feature request related to a problem? Please describe.

Currently, Moveit Servo does not work with intraprocess communication due to a missing support of KEEP_ALL QoS in rclcpp (https://github.com/ros2/rclcpp/issues/752). Therefore, the usage was disabled (https://github.com/ros-planning/moveit2/pull/722).

Describe the solution you'd like As soon as https://github.com/ros2/rclcpp/issues/752 is fixed, the changes in https://github.com/ros-planning/moveit2/pull/722 and https://github.com/ros-planning/moveit2/pull/2752 should be reverted to enable the intraprocess communication.

Describe alternatives you've considered Only alternative would be changing the code of MoveIt so that it does not use this QoS.

github-actions[bot] commented 2 months ago

This issue is being labeled as stale because it has been open 45 days with no activity. It will be automatically closed after another 45 days without follow-ups.

github-actions[bot] commented 4 weeks ago

This issue was closed because it has been stalled for 45 days with no activity.