Skip to content

Add implementation of creatind ExampleRobotWrapper with recipe vectors#456

Open
urfeex wants to merge 2 commits intoUniversalRobots:masterfrom
urfeex:add_example_robot_wrapper_with_recipe_vector
Open

Add implementation of creatind ExampleRobotWrapper with recipe vectors#456
urfeex wants to merge 2 commits intoUniversalRobots:masterfrom
urfeex:add_example_robot_wrapper_with_recipe_vector

Conversation

@urfeex
Copy link
Member

@urfeex urfeex commented Mar 5, 2026

The signature was added earlier, but the implementation was missing.

I missed that the implementation was missing when reviewing #408.

The signature was added earlier, but the implementation was missing
@urfeex urfeex requested a review from a team March 5, 2026 11:17
@urfeex urfeex added the bugfix label Mar 5, 2026
@cursor
Copy link

cursor bot commented Mar 5, 2026

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on March 20.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@codecov
Copy link

codecov bot commented Mar 5, 2026

Codecov Report

❌ Patch coverage is 92.59259% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.94%. Comparing base (c2e7fe5) to head (c799510).

Files with missing lines Patch % Lines
src/example_robot_wrapper.cpp 92.59% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #456      +/-   ##
==========================================
- Coverage   76.36%   72.94%   -3.43%     
==========================================
  Files         105      104       -1     
  Lines        5539     5437     -102     
  Branches      593      592       -1     
==========================================
- Hits         4230     3966     -264     
- Misses       1011     1179     +168     
+ Partials      298      292       -6     
Flag Coverage Δ
start_ursim 83.21% <ø> (-0.85%) ⬇️
ur20-latest 68.90% <92.59%> (+0.16%) ⬆️
ur5-3.14.3 68.65% <92.59%> (-3.29%) ⬇️
ur5e-10.11.0 62.15% <96.00%> (-2.73%) ⬇️
ur5e-10.12.0 63.12% <96.00%> (-3.11%) ⬇️
ur5e-10.7.0 61.16% <92.00%> (-3.11%) ⬇️
ur5e-5.9.4 69.30% <92.59%> (-3.13%) ⬇️

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.

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.

1 participant