robotology / human-dynamics-estimation

Software repository for estimating human dynamics
BSD 3-Clause "New" or "Revised" License
81 stars 28 forks source link

Fix joint limits read #257

Closed lrapetti closed 3 years ago

lrapetti commented 3 years ago

Currently, there is a bug in the joint limits that are passed to the stateIntegrator. The error is due to the fact that we are iterating on DOFs but then the output is used for reading the joint. The issue should be fixed by this PR.

In addition, I am fixing the iCub3 teleoperation model on the right wrist (see https://github.com/robotology/icub-models/issues/93)

lrapetti commented 3 years ago

Thanks @S-Dafarra.

Let's recall that https://github.com/robotology/human-dynamics-estimation/pull/257/commits/2a1fba7d5a7b8c7596141d2c91f141e865474094 is a temporary patch, and we should check again after the bug is fixed on the upstream model (https://github.com/robotology/icub-models/issues/93)