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

Change FT sensors names for compatibility with icub-models >= 2.0.0, robots-configuration >= 2.5.0 and ergocub-software >= 0.3.4 #720

Merged
merged 3 commits into from
Sep 4, 2023

Conversation

traversaro
Copy link
Collaborator

Fix robotology/icub-models-generator#242 .

icub-models 2.0.0 changed the name of the FT sensors in the URDF from being named <identifier>_ft_sensor (like l_arm_ft_sensor, l_leg_ft_sensor, ...) to <identifier>_ft (like l_arm_ft, l_leg_ft, ...).
However, the yarprobotinterface configuration files continued to refer to the sensors as <identifier>_ft_sensor, creating errors for software that was trying to match sensors find in URDF and sensors as exposed by the YARP's multipleanalogsensorsserver device.

This PR changes all the instances of iCub configuration files to <identifier>_ft, restoring compatibility with icub-models 2.0.0, robots-configuration releases >= 2.5.0 and ergocub-software >= 0.3.4, see robotology/robots-configuration#562 .

Note that the name of the joint to which the sensor is attached remained <identifier>_ft_sensor, both before and after icub-models 2.0.0 release.

…robots-configuration >= 2.5.0 and ergocub-software >= 0.3.4
@GiulioRomualdi
Copy link
Member

Thank you @traversaro. I will merge with squash

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Align icub-models FT sensor names with sensor names exposed by YARP devices on the real robot
3 participants