Add 10.13 to test matrix#517
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #517 +/- ##
==========================================
- Coverage 78.85% 78.72% -0.13%
==========================================
Files 116 116
Lines 6612 6610 -2
Branches 2920 2920
==========================================
- Hits 5214 5204 -10
- Misses 1031 1041 +10
+ Partials 367 365 -2 Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. |
|
@urfeex It seems the 10.13.0 is timing out in the CI! |
Ah, that's probably because of the UR15 robot model I chose there. The motion sequence contains linear pose targets that might not be applicable to a UR15. I checked with a UR5e locally. I'll check things again. There also seem to be a failing tests regarding the robot-api that we need to address. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
Reviewed by Cursor Bugbot for commit 5cbb8a3. Configure here.
| PROGRAM_FOLDER: 'tests/resources/dockerursim/programs/polyscopex' | ||
| - ROBOT_MODEL: 'ur15' | ||
| URSIM_VERSION: '10.13.0' | ||
| PROGRAM_FOLDER: 'tests/resources/dockerursim/programs/polyscopex' |
There was a problem hiding this comment.
UR15 breaks full integration job
Medium Severity
The new 10.13.0 matrix entry uses ur15 while other PolyScope X rows in the same run_tests job use ur5e, yet it still runs the full ctest suite and run_examples.sh against the shared polyscopex program folder. That mismatch aligns with reported CI timeouts when example motion (e.g. linear poses) is not valid for UR15.
Reviewed by Cursor Bugbot for commit 5cbb8a3. Configure here.


Run integration tests on 10.13, as well.
Note
Low Risk
Changes are limited to CI matrix expansion, test fixtures, and integration-test timing; no production library behavior is modified.
Overview
Adds URSim 10.13.0 coverage to CI by running the integration test job with
ur15and the PolyScope X program folder.Supporting changes reduce flakes on newer simulators:
program_interactionnow pollscommandGetLoadedProgram()for up to 1s after load instead of asserting immediately. Theupdate_prog.urpxfixture renames the application toupdate_application.Reviewed by Cursor Bugbot for commit 5cbb8a3. Bugbot is set up for automated code reviews on this repo. Configure here.