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

Suggestion: Inject the remote's IP address into the script code #25

Open
fmauch opened this issue Dec 10, 2021 · 0 comments
Open

Suggestion: Inject the remote's IP address into the script code #25

fmauch opened this issue Dec 10, 2021 · 0 comments

Comments

@fmauch
Copy link
Collaborator

fmauch commented Dec 10, 2021

Currently, we use the configured remote's IP address only for accessing the script_sender port. Our protocol specification from the client library requires two additional ports, the reverse port and the trajectory port.

The IP address of those get injected into the script code sent from the remote side, where it either gets automatically deduced from the interface that connects to the RTDE port if no particular address is passed in (see this line).

This complicates setup a bit, as users have to specify their remote PC's address twice. Once in the URCap installation and once when starting the client application. An example using the docker simulation:

docker run --rm -it -p 5900:5900 -p 6080:6080 -p 29999:29999 -p 30001:30001 -p 30004:30004 -p 30002:30002 universalrobots/ursim_e-series

This will bind the interfaces from URSim to localhost and therefore the default reverse IP used in the client library is 127.0.0.1. However, from inside the docker container this will not correspond to the host machine. For the reverse interface we want to connect to <host_ip>:50001 by default.

With saving the Installation remote's IP into the script code we can reuse that inside the actual script code instead of replacing it with the host ip on the remote side.

I'm not feeling completely happy with this, as this introduces a syntax dependency between the URCap and the client library, but it would definitely reduce the user's setup requirements.

Related issues from the drivers:

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

No branches or pull requests

1 participant