Skip to content

Added ScriptCommandInterface command setTcpOffset#444

Merged
urfeex merged 6 commits intoUniversalRobots:masterfrom
dkar-sto:feat/set_tcp_offset
Feb 25, 2026
Merged

Added ScriptCommandInterface command setTcpOffset#444
urfeex merged 6 commits intoUniversalRobots:masterfrom
dkar-sto:feat/set_tcp_offset

Conversation

@dkar-sto
Copy link
Contributor

@dkar-sto dkar-sto commented Feb 19, 2026

simply calls the urscript command set_tcp()


Note

Medium Risk
Adds a new control-plane command that changes the robot TCP configuration at runtime; incorrect offsets or misuse could impact motion accuracy. Changes are otherwise small and covered by new unit/integration tests.

Overview
Adds a new ScriptCommandInterface command (SET_TCP_OFFSET) and URScript handling to call set_tcp() with a provided [x, y, z, rx, ry, rz] offset.

Exposes this as UrDriver::setTcpOffset(), updates the command enum/protocol constant, makes client_connected_ atomic, and adds new tests covering message encoding and an end-to-end RTDE validation of tcp_offset (including failure when the external control program is not running).

Written by Cursor Bugbot for commit 88c0a13. This will update automatically on new commits. Configure here.

@codecov
Copy link

codecov bot commented Feb 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.27%. Comparing base (4dd918b) to head (88c0a13).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #444      +/-   ##
==========================================
+ Coverage   76.22%   76.27%   +0.04%     
==========================================
  Files         104      104              
  Lines        5509     5529      +20     
  Branches      590      593       +3     
==========================================
+ Hits         4199     4217      +18     
- Misses       1012     1014       +2     
  Partials      298      298              
Flag Coverage Δ
start_ursim 82.54% <ø> (-0.73%) ⬇️
ur20-latest 68.76% <100.00%> (+0.18%) ⬆️
ur5-3.14.3 71.72% <100.00%> (+0.06%) ⬆️
ur5e-10.11.0 64.58% <100.00%> (-0.13%) ⬇️
ur5e-10.12.0 66.08% <100.00%> (+0.08%) ⬆️
ur5e-10.7.0 63.97% <100.00%> (-0.13%) ⬇️
ur5e-5.9.4 72.45% <100.00%> (+<0.01%) ⬆️

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

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

dkar-sto and others added 2 commits February 20, 2026 06:41
@urfeex urfeex force-pushed the feat/set_tcp_offset branch from 2d27df9 to 8dd225c Compare February 24, 2026 14:25
@urfeex urfeex force-pushed the feat/set_tcp_offset branch from 8dd225c to 88c0a13 Compare February 24, 2026 14:28
Copy link

@cursor cursor bot left a comment

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 and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@urfeex urfeex added the enhancement New feature or request label Feb 25, 2026
Copy link
Member

@urfeex urfeex left a comment

Choose a reason for hiding this comment

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

Thank you for implementing this. I took the liberty of adding a UrDriver test, as well.

@urfeex urfeex merged commit 7b57b66 into UniversalRobots:master Feb 25, 2026
47 of 58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants