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 FT IMU in the ergoCubGazeboV1* models #216

Closed
GiulioRomualdi opened this issue Feb 12, 2024 · 7 comments · Fixed by #217
Closed

Add FT IMU in the ergoCubGazeboV1* models #216

GiulioRomualdi opened this issue Feb 12, 2024 · 7 comments · Fixed by #217
Labels
domain-software Related to Software prj-ergocub Related to ErgoCub Project team-fix Related to Team Fix

Comments

@GiulioRomualdi
Copy link
Contributor

Description of the Task

The robot's Force/Torque (FT) sensor is equipped with an Inertial Measurement Unit (IMU) capable of streaming orientation data. Integrating this feature into simulation would be beneficial.

Previously, @G-Cervettini initiated a Pull Request (PR) to integrate this functionality, #145. However, it involved manual modifications to the urdf file.

Definition of Completion

Successfully implement the IMU streaming within the Gazebo model using multipleanalogsensorserver.

@GiulioRomualdi GiulioRomualdi added domain-software Related to Software prj-ergocub Related to ErgoCub Project team-fix Related to Team Fix labels Feb 12, 2024
@Nicogene
Copy link
Member

@martinaxgloria also will benefit of this change

@GiulioRomualdi
Copy link
Contributor Author

I'm going to open a PR for the feet imu (added with xml blob)

@Nicogene
Copy link
Member

I'm going to open a PR for the feet imu (added with xml blob)

Why not as sensors in the yaml? Using the same CSYS of the ft gives problems?

@GiulioRomualdi
Copy link
Contributor Author

I've never did it. Can you point me some line of the code I can use as inspiration?

@GiulioRomualdi
Copy link
Contributor Author

Probably something similar to

https://github.com/icub-tech-iit/ergocub-software/blob/master/urdf%2Fcreo2urdf%2Fdata%2Fergocub1_0%2FERGOCUB_all_options.yaml#L808

@GiulioRomualdi
Copy link
Contributor Author

If I'm not mistaken, @G-Cervettini noticed that the imu plugin needs to be attached to a link, and currently, there are no links exposed in the FTs. (The FT plugins are connected to a fixed joint) For this reason @G-Cervettini manually specified the pose of the sensor with respect to the parent. I don't know if this is possible with the approach you propose (for the head imu the pose seems to be zero)

@Nicogene
Copy link
Member

At the end in :

We went for adding as xml blobs because we spotted a bug in creo2urdf:

The generated imu sensors had wrong poses, in simmecanics_to_urdf seems not to be a problem exporting an ft and imu that refers to the same csys:

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.

2 participants