Skip to content

Add 10.13 to test matrix#517

Merged
urfeex merged 4 commits into
UniversalRobots:masterfrom
urfeex:test_10.13
Jun 26, 2026
Merged

Add 10.13 to test matrix#517
urfeex merged 4 commits into
UniversalRobots:masterfrom
urfeex:test_10.13

Conversation

@urfeex

@urfeex urfeex commented Jun 10, 2026

Copy link
Copy Markdown
Member

Run integration tests on 10.13, as well.


Note

Low Risk
Changes are limited to CI matrix expansion, test fixtures, and integration test polling; no production library behavior is modified.

Overview
Adds URSim 10.13.0 integration coverage for ur7e with the PolyScope X program folder in the CI run_tests matrix.

Supporting tweaks for the new simulator version: the PolyScope X fixture update_prog.urpx renames the embedded application from application to update_application, and program_interaction no longer asserts loaded program name immediately after commandLoadProgram—it polls commandGetLoadedProgram for up to 1s until the name is wait_program, avoiding flakes when load is asynchronous on newer PolyScope X.

Reviewed by Cursor Bugbot for commit 9a1f505. Bugbot is set up for automated code reviews on this repo. Configure here.

@urfeex urfeex requested a review from a team June 10, 2026 11:46
@urfeex urfeex added the CI label Jun 10, 2026
@codecov

codecov Bot commented Jun 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.91%. Comparing base (e01f5f3) to head (9a1f505).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #517      +/-   ##
==========================================
- Coverage   78.98%   78.91%   -0.07%     
==========================================
  Files         115      115              
  Lines        6765     6764       -1     
  Branches     2986     2986              
==========================================
- Hits         5343     5338       -5     
- Misses       1052     1053       +1     
- Partials      370      373       +3     
Flag Coverage Δ
check_version_ur10-3.15.8 11.11% <ø> (-1.46%) ⬇️
check_version_ur10e-10.11.0 11.11% <ø> (-0.05%) ⬇️
check_version_ur10e-5.15.2 11.16% <ø> (-0.19%) ⬇️
check_version_ur12e-10.12.1 11.11% <ø> (-0.05%) ⬇️
check_version_ur12e-5.25.1 11.16% <ø> (-0.19%) ⬇️
check_version_ur15-10.12.1 11.11% <ø> (-0.05%) ⬇️
check_version_ur15-5.25.1 11.11% <ø> (-0.19%) ⬇️
check_version_ur16e-10.12.1 11.11% <ø> (ø)
check_version_ur16e-5.25.1 11.16% <ø> (+0.04%) ⬆️
check_version_ur18-10.12.1 11.11% <ø> (ø)
check_version_ur18-5.25.1 11.11% <ø> (-0.24%) ⬇️
check_version_ur20-10.12.1 11.11% <ø> (-0.05%) ⬇️
check_version_ur20-5.25.1 11.30% <ø> (+0.18%) ⬆️
check_version_ur3-3.14.3 11.52% <ø> (-0.52%) ⬇️
check_version_ur30-10.12.1 11.16% <ø> (ø)
check_version_ur30-5.25.1 11.11% <ø> (ø)
check_version_ur3e-10.11.0 11.11% <ø> (ø)
check_version_ur3e-5.9.4 11.16% <ø> (ø)
check_version_ur5-3.15.8 11.16% <ø> (-0.50%) ⬇️
check_version_ur5e-10.11.0 11.11% <ø> (ø)
check_version_ur5e-5.12.8 11.16% <ø> (ø)
check_version_ur7e-10.11.0 11.11% <ø> (ø)
check_version_ur7e-5.22.2 11.11% <ø> (-0.77%) ⬇️
check_version_ur8long-10.12.1 11.20% <ø> (+0.04%) ⬆️
check_version_ur8long-5.25.1 11.11% <ø> (-0.19%) ⬇️
python_scripts 75.90% <ø> (ø)
start_ursim 83.63% <ø> (-1.51%) ⬇️
ur5-3.14.3 74.47% <ø> (ø)
ur5e-10.11.0 69.23% <ø> (-0.02%) ⬇️
ur5e-10.12.0 70.30% <ø> (-0.05%) ⬇️
ur5e-10.7.0 68.54% <ø> (-0.15%) ⬇️
ur5e-5.9.4 75.13% <ø> (-0.11%) ⬇️
ur7e-10.13.0 70.24% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@urrsk

urrsk commented Jun 12, 2026

Copy link
Copy Markdown
Member

@urfeex It seems the 10.13.0 is timing out in the CI!

@urfeex

urfeex commented Jun 12, 2026

Copy link
Copy Markdown
Member Author

@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.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

Reviewed by Cursor Bugbot for commit 5cbb8a3. Configure here.

Comment thread .github/workflows/ci.yml
@urrsk

urrsk commented Jun 25, 2026

Copy link
Copy Markdown
Member

@urfeex Let's change the type to a UR7 and get it in.
When we can change the position in the example to also let the UR15 able to run it

@mergify

mergify Bot commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@urfeex urfeex merged commit 171b109 into UniversalRobots:master Jun 26, 2026
52 of 64 checks passed
@urfeex urfeex deleted the test_10.13 branch June 26, 2026 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants