moveit / moveit2

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

Why does Trajectory Execution Manager use MultiThreadedExecutor? #2880

Closed CihatAltiparmak closed 6 days ago

CihatAltiparmak commented 3 months ago

Description

While trying to integrate rmw_zenoh to MoveIt, i have seen that MoveGroup server uses MultiThreadedExecutor. rmw_zenoh doesn't work at the moment because of MultiThreadedExecutor. My aim here is not to report a bug, but to start a discussion about whether it's really necessary to use MultiThreadedExecutor here.

https://github.com/moveit/moveit2/blob/94796046b5ae6fc2a8e5f5ee73db4944d71a5643/moveit_ros/planning/trajectory_execution_manager/src/trajectory_execution_manager.cpp#L193-L201

https://github.com/moveit/moveit2/blob/94796046b5ae6fc2a8e5f5ee73db4944d71a5643/moveit_ros/move_group/src/move_group.cpp#L312-L327

Your environment

github-actions[bot] commented 1 month 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 6 days ago

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