rclcpp icon indicating copy to clipboard operation
rclcpp copied to clipboard
verified publisher icon ros2

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

Open wjwwood opened this issue 1 year ago • 4 comments

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 avatar Jun 12 '24 21:06 wjwwood

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

wjwwood avatar Jun 19 '24 00:06 wjwwood

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

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

wjwwood avatar Jun 20 '24 21:06 wjwwood

CI after uncrustify fix:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

wjwwood avatar Jun 25 '24 20:06 wjwwood

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.

wjwwood avatar Jun 25 '24 22:06 wjwwood

Since this is targeting Iron, which is going EOL soon, I'm going to close this out.

@wjwwood if you think it makes sense to retarget this to rolling, please feel free to reopen and retarget.

clalancette avatar Nov 05 '24 18:11 clalancette