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

[ergoCubSN001] Disable FT sensors in URDF #200

Closed
pattacini opened this issue Dec 6, 2023 · 11 comments · Fixed by #206
Closed

[ergoCubSN001] Disable FT sensors in URDF #200

pattacini opened this issue Dec 6, 2023 · 11 comments · Fixed by #206
Assignees
Labels
domain-software Related to Software prj-ergocub Related to ErgoCub Project team-fix Related to Team Fix

Comments

@pattacini
Copy link
Member

pattacini commented Dec 6, 2023

Task description

Following up on robotology/robots-configuration#598 (comment), we're requested to update the URDF of ergoCubSN001 to disable the FT sensors.

Definition of Done

  • URDF modified accordingly
  • URDF tested in simulation

cc @Nicogene @GiulioRomualdi

@pattacini pattacini added domain-software Related to Software prj-ergocub Related to ErgoCub Project team-fix Related to Team Fix labels Dec 6, 2023
@Nicogene
Copy link
Member

Nicogene commented Dec 7, 2023

The arms FTs seems already there:

<gazebo reference="l_arm_ft_sensor">
<sensor name="l_arm_ft" type="force_torque">
<always_on>1</always_on>
<update_rate>100</update_rate>
<force_torque>
<frame>sensor</frame>
<measure_direction>child_to_parent</measure_direction>
</force_torque>
<pose>-3.46945e-18 0 0 3.14159 1.94289e-16 -1.5708 </pose>
<plugin name="left_arm_ft_plugin" filename="libgazebo_yarp_forcetorque.so">
<yarpConfigurationFile>model://ergoCub/conf/FT/gazebo_ergocub_left_arm_ft.ini</yarpConfigurationFile>
</plugin>
</sensor>
</gazebo>

<gazebo reference="r_arm_ft_sensor">
<sensor name="r_arm_ft" type="force_torque">
<always_on>1</always_on>
<update_rate>100</update_rate>
<force_torque>
<frame>sensor</frame>
<measure_direction>child_to_parent</measure_direction>
</force_torque>
<pose>0 0 0 -3.14159 -2.77556e-17 1.5708 </pose>
<plugin name="right_arm_ft_plugin" filename="libgazebo_yarp_forcetorque.so">
<yarpConfigurationFile>model://ergoCub/conf/FT/gazebo_ergocub_right_arm_ft.ini</yarpConfigurationFile>
</plugin>
</sensor>
</gazebo>

@GiulioRomualdi which ones are missing?

@AntonioConsilvio
Copy link
Member

AntonioConsilvio commented Dec 7, 2023

Hi @Nicogene @pattacini, I think there was a little misunderstanding. The FTs on the shoulders have been disabled from configuration files due to malfunctioning, so the sensors should be disabled from the URDF to be compatible with the configuration files.

@pattacini pattacini changed the title [ergoCubSN001] Reenable FT sensors in URDF [ergoCubSN001] Disable FT sensors in URDF Dec 7, 2023
@pattacini
Copy link
Member Author

pattacini commented Dec 7, 2023

Ok thanks @AntonioConsilvio for the heads-up 👍🏻
My bad!
I've updated the OP and the title accordingly.

@Nicogene Nicogene self-assigned this Dec 7, 2023
@Nicogene
Copy link
Member

Nicogene commented Dec 7, 2023

@GiulioRomualdi do you need this change in ergoCubSN001 or/and ergoCubGazeboV1_1 urdf?

@Nicogene
Copy link
Member

Nicogene added a commit that referenced this issue Dec 20, 2023
@GiulioRomualdi
Copy link
Contributor

GiulioRomualdi commented Dec 20, 2023

@GiulioRomualdi do you need this change in ergoCubSN001 or/and ergoCubGazeboV1_1 urdf?

Sorry, I completely missed the message :(
We need to remove them from ergoCubSN001 no need from ergoCubGazeboV1_1

@Nicogene
Copy link
Member

Nicogene commented Dec 20, 2023

@GiulioRomualdi do you need this change in ergoCubSN001 or/and ergoCubGazeboV1_1 urdf?

Sorry, I completely missed the message :( We need to remove them from ergoCubSN001 no need from ergoCubGazeboV1_1

I am afraid that I have to remove it also from ergoCubGazeboV1_1 since I have to remove the nws for those ft from the xml that is common between all the models.

But if the model ergoCubSN001 is used without launghing the yarprobotinterface plugin I can remove them only from that urdf and keep the ergocub.xml unchanged.

cc @traversaro

@GiulioRomualdi
Copy link
Contributor

In this case i think we should update wbd xml file. Are there 2 separate files for gazebov1 and gazebov1_1?

@traversaro
Copy link
Member

Related to robotology/whole-body-estimators#32, that should simplify doing this.

@Nicogene
Copy link
Member

In this case i think we should update wbd xml file. Are there 2 separate files for gazebov1 and gazebov1_1?

Unfortunately not, it was not needed until now, but I would avoid adding an xml file for this temporary situation

Nicogene added a commit that referenced this issue Dec 28, 2023
@Nicogene
Copy link
Member

Nicogene commented Jan 8, 2024

They have been re-enabled:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain-software Related to Software prj-ergocub Related to ErgoCub Project team-fix Related to Team Fix
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants