Skip to content
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

Add doc to custom URScript commands that it needs to be in headless mode #1051

Merged
merged 3 commits into from
Jul 15, 2024

Conversation

urrsk
Copy link
Member

@urrsk urrsk commented Jul 10, 2024

Seems to be an important detail missing that the custom URScript command only works in headless mode

@urrsk urrsk requested a review from URJala July 10, 2024 09:00
@@ -221,6 +223,7 @@ restarted again. Depending whether you use headless mode or not, you'll have to
external_control program again.

.. note::
Custom URScript command only works in headless mode. And does not work with the external_control URCap.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think, this is a bit misleading, as in fact it is working together with the external_control URCap. The program just has to be restarted afterwards, as stated above.

Copy link
Member Author

Choose a reason for hiding this comment

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

@fmauch No that is not correct. On the e-series it only will work when the robot in remote control.
Maybe that is what we should write

ur_robot_driver/doc/usage.rst Outdated Show resolved Hide resolved
ur_robot_driver/doc/usage.rst Outdated Show resolved Hide resolved
Co-authored-by: Felix Exner (fexner) <felix_mauch@web.de>
@urrsk urrsk requested a review from fmauch July 10, 2024 13:30
Copy link
Contributor

@fmauch fmauch 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 @urrsk!

@fmauch fmauch merged commit ec79784 into UniversalRobots:main Jul 15, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants