ami-iit / yarp-openmct

Repo for YARP and OpenMCT integration.
BSD 3-Clause "New" or "Revised" License
6 stars 1 forks source link

Visualise inertials (from all IMUs) #134

Closed nunoguedelha closed 2 years ago

nunoguedelha commented 2 years ago

Implements #124 .

nunoguedelha commented 2 years ago

Hi @traversaro , I have a doubt about the torso data format (/icub/torso/inertialGYRO), I have to check with @S-Dafarra , but the changes are already reviewable.

nunoguedelha commented 2 years ago

Ok for me, moduling the parsing that probably is blocked by understanding the semantics of the data.

After discussing with @S-Dafarra , we decided to drop the torso "inertialGYRO" configuration since there is no actual IMU on the torso.

Pushing a commit shortly. I'll merge the PR directly without a new approval if that's ok for you.

nunoguedelha commented 2 years ago

Pushing a commit shortly. I'll merge the PR directly without a new approval if that's ok for you.

Actually, it's simpler to just push the change in the latest ongoing PR #136 .