-
Notifications
You must be signed in to change notification settings - Fork 71
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 non collocated external wrenches estimation Berdy variant #991
Add non collocated external wrenches estimation Berdy variant #991
Conversation
5eb207f
to
cd01ed7
Compare
|
f855f9d
to
5811248
Compare
cc @traversaro |
a889084
to
c6c4d1c
Compare
cc489f8
to
6d0e015
Compare
@traversaro the PR is ready to be reviewed. I suppose that the one check that is not completed is due to the fact that the commit where the matlab bindings have been regenerated are authored by a github action so we can ignore it. |
Thanks, this seems nice! Remaining points:
|
Thanks, I didn't think about that. Added in 8dab362.
You can proceed with "Squash and merge"! |
This PR adds a new variant to the Berdy MAP Estimator for estimating the external wrenches acting on a multi-rigid body, which is the relation (25b) of https://ieeexplore.ieee.org/abstract/document/9526592.
The PR adds a new fictitious sensor for the Rate of Change of Momentum (RCM), which provides a new measurement for the berdy problem.
The Rate of Change of Momentum (RCM) should be computed in the CoM as per https://ieeexplore.ieee.org/abstract/document/9526592, but here the adjoint wrench transforms related to the RCM are not from the i-th link to the centroidal frame, but to the base one. For this reason
Y*d
will compute the sum of the estimated external wrenches transformed in the base frame, not the centroidal one.For this reason, the measurement of the fictitious sensor should account for the bias terms generated by this transformation.