robocup-software icon indicating copy to clipboard operation
robocup-software copied to clipboard

Backport `spin_all` and use it for temporary executors.

Open kylestach opened this issue 5 years ago • 1 comments

Currently we are using spin_some or spin() in a separate thread for our temporary executors. This should technically really be replaced with a spin_all(), which has been added to rclcpp master but has not yet been added to the foxy release.

Pull request: https://github.com/RoboJackets/robocup-software/pull/1554

Link to TODO, or file number/line: https://github.com/RoboJackets/robocup-software/blob/1aa4cee694700dd91c4fbb1eb343c8dc772a50fe/soccer/src/soccer/processor.cpp#L146

kylestach avatar Aug 01 '20 22:08 kylestach

The following code from rclcpp will be helpful:

https://github.com/ros2/rclcpp/blob/aa159a5e8f6d3bf50c4c0c40dbfb656f7e8c6f97/rclcpp/src/rclcpp/executor.cpp#L406-L450

kylestach avatar Oct 29 '20 21:10 kylestach