roboticslab-uc3m / teo-main

TEO full-sized humanoid robot: super/meta repository.
http://roboticslab.uc3m.es/roboticslab/robot/teo-humanoid
GNU Lesser General Public License v2.1
4 stars 1 forks source link

Review and Unify TEO joint names #44

Open jgvictores opened 6 years ago

jgvictores commented 6 years ago

Review and Unify TEO joint names

jgvictores commented 6 years ago

For reference, I'd use what is currently called Simox-names from: https://github.com/roboticslab-uc3m/teo-developer-manual/blob/8d2539261ca1755c7040958b9384d99af10dfbe4/assets/motores-motores.csv

jgvictores commented 5 years ago

Some refs, possibly not too relevant:

PeterBowman commented 5 years ago

Note: "saggital, frontal, axial..." might be misplaced, e.g. current saggital leg joints are actually frontal joints according to https://en.wikipedia.org/wiki/Anatomical_plane.

jgvictores commented 4 years ago

ASWB:

jgvictores commented 4 years ago

Related: https://github.com/roboticslab-uc3m/teo-developer-manual/issues/62

PeterBowman commented 4 years ago

Note: "saggital, frontal, axial..." might be misplaced, e.g. current saggital leg joints are actually frontal joints according to https://en.wikipedia.org/wiki/Anatomical_plane.

Speaking of planes, yes. Speaking of axes, cf. https://www.researchgate.net/publication/270340667 and check the following diagram:

Anatomical-planes-of-the-upper-region-of-the-human-body-1_W640

More documents seem to avail this choice of names, see e.g. this one:

The sagital axis passes horizontally from posterior to anterior and is formed by the intersection of the sagital and transverse planes.

The frontal axis passes horizontally from left to right and is formed by the intersection of the frontal and transverse planes.

The vertical axis passes vertically from inferior to superior and is formed by the intersection of the sagital and frontal planes.

jgvictores commented 4 years ago

I guess this means we could actually keep something similar to the current names? Namely, regarding root frame:

Therefore: FrontalLeftElbow, FrontalTrunk, right?

jgvictores commented 4 years ago

PS: I've also thought about simply LeftElbowY (LEY) and TrunkY (TY). Partially discarded as these refer to root frame and change depending on previous rotations (but same reasoning may apply to above...).

PeterBowman commented 4 years ago

I think the sagittal/frontal/axial naming scheme is well established and, since it's actually correct (contrary to what we thought), I'd stick to it rather than switching to XYZ suffixes.

Therefore: FrontalLeftElbow, FrontalTrunk, right?

OK.

PeterBowman commented 4 years ago

Note: review and unify OpenRAVE joint names, too.

jgvictores commented 4 years ago

Note: review and unify OpenRAVE joint names, too.

Updated issue description. May have added some overkill: for now our .ini's do not contain this sort of info.

jgvictores commented 4 years ago

Done at https://github.com/roboticslab-uc3m/teo-developer-manual/commit/e6475723b484c020d46d6251c43d2c61822db6ee

rsantos88 commented 4 years ago

Done at https://github.com/roboticslab-uc3m/teo-openrave-models/pull/35

jgvictores commented 4 years ago

Done at roboticslab-uc3m/teo-openrave-models#35

Perfect! I added the PR related to the OpenRAVE model to the TEO project for visibility, but keeping this issue open for tracking other models. Thanks!!!