robotology / idyntree

Multibody Dynamics Library designed for Free Floating Robots
BSD 3-Clause "New" or "Revised" License
177 stars 67 forks source link

Implement TorqueEstimation without depending on KDL #97

Closed traversaro closed 8 years ago

traversaro commented 8 years ago

Similarly to https://github.com/robotology/idyntree/issues/93 , this meta-issue is useful to track what to we miss to perform torque estimation (i.e. wholeBodyDynamicsTree) using only iDynTree data structures. (As for #93 this is not a big priority, but it would definitely simplify the use of the wholeBodyDynamicsTree/wholeBodyEstimator for all the people using just the iCub infrastructure and not the CoDyCo one, see also https://github.com/robotology/codyco-modules/issues/158 ):

naveenoid commented 8 years ago

Can i handle one or more of the sub tasks? Is there any task assignments so far?

traversaro commented 8 years ago

Actually this task has quite a low priority (it is not blocking anything else, and the current KDL-based TorqueEstimation class is working fine) so the task have not be assigned to anyone, but if you like feel free to work on this.

traversaro commented 8 years ago

Just to clarify: this issue is tracking how to implement in iDynTree the algorithm already implemented in the TorqueEstimation . It is not tracking how to implement the torque estimation using berdy.

traversaro commented 8 years ago

@nunoguedelha I added some docs to the data structures currently used for external wrenches computations in https://github.com/robotology/idyntree/commit/5973ce7608c67640806046a28fab720c1d8cd93a .

As most of the dynamics function are currently not documented (documenting all of them is definitely is a desirable goal, but it would take some time) @nunoguedelha can you tell me which one do you prefer that I document because their use/semantics is not clear?

traversaro commented 8 years ago

The tricky part is that we currently have the following data structure for modeling external forces :

The use of dynContactList is (partially) documented in this part of the icub wiki : http://wiki.icub.org/wiki/Tactile_sensors_%28aka_Skin%29#Tactile_Data_Flow .

traversaro commented 8 years ago

Other interesting documents are :

nunoguedelha commented 8 years ago

my last commit on current implementation here: https://github.com/robotology/idyntree/tree/feature/97/contactListWoKDL

traversaro commented 8 years ago

See https://github.com/robotology/idyntree/pull/130 . Issue update to link the relevant commits for each bullet point.