Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Simulated finger hall effect sensors for iCubGazeboV2_5_visuomanip have wrong port names #214

Closed
xEnVrE opened this issue Oct 18, 2023 · 1 comment · Fixed by #215
Closed

Comments

@xEnVrE
Copy link
Contributor

xEnVrE commented Oct 18, 2023

As per the title, the ports that the system open are as follows

registration name /iCub/left_hand/analog:o ip 172.17.0.1 port 10003 type tcp
registration name /iCub/left_hand/analog:o/rpc:i ip 172.17.0.1 port 10002 type tcp
registration name /iCub/right_hand/analog:o ip 172.17.0.1 port 10005 type tcp
registration name /iCub/right_hand/analog:o/rpc:i ip 172.17.0.1 port 10004 type tcp

while they should begin with icubSim.

The problem seems to be that the following include:

is failing as the iCub_manual/conf_manual/iCubGazeboV2_5_visuomanip folder does not contain the gazebo_icub_robotname.ini.

Can I open a PR to add that file?

Or, given that file is basically unused as, for the rest, we provide the portprefix via the yarprobotinterface plugin, e.g.:

<robot name="iCubGazeboV3" build="1" portprefix="/icubSim" xmlns:xi="http://www.w3.org/2001/XInclude">

should I open a PR to hardcode icubSim within the *_mais.ini files?

Thank you

cc @traversaro

@traversaro
Copy link
Member

should I open a PR to hardcode icubSim within the *_mais.ini files?

Yes, I think that is the easiest solution.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
2 participants