-
Notifications
You must be signed in to change notification settings - Fork 80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added interface for forwarding script commands to the robot, that is … #111
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general seems very clean. Tested together with UniversalRobots/Universal_Robots_ROS_Driver#567 and works fine.
Not approving because of a couple of discussion points left.
…then executed locally on the robot This allows the user to be able to zero ftsensor, set payload and set tool voltage when the robot is in local control.
Co-authored-by: Felix Exner <felix_mauch@web.de>
… throwing an exception
8b07dca
to
4334ae0
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #111 +/- ##
=========================================
Coverage ? 40.35%
=========================================
Files ? 78
Lines ? 2166
Branches ? 265
=========================================
Hits ? 874
Misses ? 1179
Partials ? 113 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, Thanks @urmahp
…then executed locally on the robot
This allows the user to be able to zero ftsensor, set payload and set tool voltage when the robot is in local control.