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

Possible wrong position of waist IMU ergoCubSN001 #260

Closed
fils99 opened this issue Sep 11, 2024 · 13 comments · Fixed by #267
Closed

Possible wrong position of waist IMU ergoCubSN001 #260

fils99 opened this issue Sep 11, 2024 · 13 comments · Fixed by #267
Assignees
Labels
domain-software Related to Software prj-ergocub Related to ErgoCub Project team-fix Related to Team Fix

Comments

@fils99
Copy link

fils99 commented Sep 11, 2024

Task description

Today, during experiments with ergoCubSN001, we wanted to check the position and orientation of the waist IMU. First of all we looked to the IMU in the real robot. Afterwards, we compared them with the ones of the URDF model (we use this version of ergocub-software e4b567e).

shared image

As we can see in the image, the position of waist IMU in the real robot seems a bit different wrt the URDF one.

Definition of Done

Understand if position of waist IMU is correct

cc @LoreMoretti @GiulioRomualdi

@fils99 fils99 added domain-software Related to Software prj-ergocub Related to ErgoCub Project team-fix Related to Team Fix labels Sep 11, 2024
@GiulioRomualdi
Copy link
Contributor

cc @Nicogene and @traversaro

@Nicogene
Copy link
Member

The imu in the waist has been added in the SN000 urdf as XML blob (since the simulation model cad is unmaintained):

I didn't recall that is still exported as XML blob also in newer models:

- |
<link name="waist_imu_0"/>
- |
<joint name="waist_imu_0_fixed_joint" type="fixed">
<origin xyz="0.0354497 0.00639688 0.060600" rpy="1.5708 0 -1.5708"/>
<parent link="torso_1"/>
<child link="waist_imu_0"/>
<dynamics damping="0.1"/>
</joint>

Maybe @Mick3Lozzo recall why we didn't exported that frame, but we should do it from the cad (cc @fiorisi @Lawproto)

@Nicogene Nicogene self-assigned this Sep 11, 2024
@fiorisi
Copy link
Member

fiorisi commented Sep 11, 2024

If I correctly understood the problem I think that the IMU was moved because of a possible collision with the torso covers: https://github.com/icub-tech-iit/project-ergocub/issues/107#issuecomment-1531510392.

@Nicogene
Copy link
Member

If I correctly understood the problem I think that the IMU was moved because of a possible collision with the torso covers: icub-tech-iit/project-ergocub#107 (comment).

Good catch!

@Nicogene
Copy link
Member

Nicogene commented Oct 2, 2024

Today I checked the transform between the SCSYS_TORSO_1 and the waist IMU csys and here is the result

Image

I used this snippet using the idyntree binding for getting xyzrpy:


rpy = idyntree.bindings.Rotation(0, 0, -1, -1, 0.000105208, 0, 0.000105208, 1, 0).asRPY()

And I got:

-<origin xyz="0.0354497 0.00639688 0.060600" rpy="1.5708 0 -1.5708"/> 
+<origin xyz="0.0606 -0.00639428 0.01745" rpy="1.5708 -0.000105208 -1.5708"/> 

The difference seems not only a displacement, I used the correct CSYS for computing the transform? Are those csys correct? I took them from ecub.asm (cc @fiorisi @Mick3Lozzo)

@Nicogene
Copy link
Member

Nicogene commented Oct 2, 2024

@Mick3Lozzo
Copy link
Member

xref:

@Nicogene what kind of IMU support is AMI using? I am going to check.

cc @fiorisi

@Nicogene
Copy link
Member

Nicogene commented Oct 2, 2024

xref:

@Nicogene what kind of IMU support is AMI using? I am going to check.

cc @fiorisi

The xsens mti that is mounted in the torso, I got that transform pulling the latest cad-mechanics

@Mick3Lozzo
Copy link
Member

The difference seems not only a displacement, I used the correct CSYS for computing the transform. Are those csys correct? I took them from ecub.asm (cc @fiorisi @Mick3Lozzo)

Image

These are the same values I found @Nicogene.

@Nicogene
Copy link
Member

Nicogene commented Oct 2, 2024

Thanks @Mick3Lozzo !
That very small rotation on pitch is strange ( we can also approximate it to zero ), but if this is the correct pose I will change the urdf accordingly

@Nicogene
Copy link
Member

Nicogene commented Oct 8, 2024

Something has to be fixed:

Probably on creo2urdf side

@Nicogene
Copy link
Member

Nicogene commented Oct 9, 2024

Hi @fils99, with:

The position of waist_imu_0 is:

Image

That seems coherent with

Image

@fils99
Copy link
Author

fils99 commented Oct 9, 2024

Thanks @Nicogene!

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