search

ros2 / rclcpp

rclcpp (ROS Client Library for C++)
Apache License 2.0
536 stars 417 forks source link

[iron] add test to check for execution order of entities in various executors #2561

Open wjwwood opened 3 months ago

wjwwood commented 3 months ago

This is the Iron version of https://github.com/ros2/rclcpp/pull/2536 and passes without changes to the actual source of rclcpp, which supports the idea that this test is actually testing a regression between iron and jazzy. The fix for jazzy/rolling has been proposed here: https://github.com/ros2/rclcpp/pull/2537

wjwwood commented 3 months ago

CI:

wjwwood commented 3 months ago

I messed up the CI, here's some hopefully fixed runs:

wjwwood commented 3 months ago

CI after uncrustify fix:

wjwwood commented 3 months ago

Unfortunately the order of execution for timers seems to be subtly out of the expected order on Windows 🤦 . I'll look into it more tonight.