-
Notifications
You must be signed in to change notification settings - Fork 104
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
Proposal for iCub naming conventions #57
Comments
👍 Nice summary! |
If we want to improve readability I would personally go for the complete prefix, like |
Nice work @traversaro. I vote +1 for I propose we put this convention in the iCub wiki extending http://eris.liralab.it/wiki/ICub_joints |
Actually: checking http://www.ros.org/reps/rep-0120.html looks like the use |
To wrap up the contributions:
[1] Romeo/Nao: https://github.com/laas/romeo/blob/master/romeo_description/urdf/romeo.urdf |
Sorry for the supporter of |
cc @luca-fiorio @johnlaser |
@johnlaser correctly pointed out that the |
... is an horrible and misleading name I would propose torso_1 e torso_2 |
Yeah, the fact that we came up with the same name indipendently mean this names are ok, implementing the changes. |
Regarding additional frame for compliance with REP-120 , we recently discussed with @hu-yue that for working on locomotion she needs a stable reference frame fixed with the sole of the robot. |
@johnlaser correctly pointed out that |
@traversaro I don't get it. For the SkinPart enum the |
Same concern as @alecive here. |
@alecive In the current proposal Notice that this mismatch is present even in port names of the skin : http://wiki.icub.org/wiki/Tactile_sensors_%28aka_Skin%29 Possible alternative names are: |
Proposal for the more important rigid links: |
I guess |
The other "important" rigid link is the |
Ok I got what you mean. Regarding the port names in the skin manager, at least for me that is a bug, especially since the Therefore, I would rather go with @iron76 's proposal (but I'd use |
|
Given the feedback received, I have prepared a new proposal for Link naming, that I report in the following. Please note that changing names involve some changes in a lot of configuration files that we use in codyco software, so reaching a consensus as soon as possible simplifies things. @alecive |
@traversaro got it . And I agree with you 👍 |
+1 Small comment: |
@lornat75 torso used as a link name could create some confusion with the joints that are named torso_jointname and the little links torso_1, torso_2 that are connecting the root_link to the chest. |
It was just a small comment. I am fine with |
I migrated the proposal that we discussed here in http://wiki.icub.org/wiki/ICub_Model_naming_conventions . |
👍 |
In the process of producing software-friendly models for the iCub robot directly from the Creo CAD models in a semi-automatically, we needed a convention for naming the different elements that constitute the robot. I will proposed a convention extending existing naming schemes, and I will be happy to get any feedback. Sharing a common convention in existing software would help to reduce incompatibilities and integration problems.
Joints
1-DOF Joints
Note: it could be useful to distinguish between the two different concepts of degree of freedom and joint. For simplicity we decide to ignore for the time being this difference, that is relevant for multi-DOF joints and closed kinematic structures.
The joints are mechanism that connect two different links (as defined in the following section) of the robot. 1-DOF joints allow motion only along one direction between the two connected links.
For joints an existing convention is introduced in http://eris.liralab.it/wiki/ICub_joints . I do not know if this names are actually used anywhere in the code and/or documentation (@barbalberto perhaps in IAxisInfo implementation?). Unfortunately in that convention the names are not unique across the robot, so for example
knee
is used to refer to both left and right knee. So we propose to use that convention, with an additionall_
orr_
prefix for joints that come in symmetric pairs. Disregarding for the moment eyes and hands, then we have the following joint names:I am personally ok with this names, but perhaps someone (@luca-fiorio ?) has some comments and may thinks that this names are misleading.
Fixed (0-DOFs) Joints
When dealing with 6-axis FT sensors, it is convenient to model them as joints that do not allow any motion between the two connected links.
In the iCub can be mounted up to 6 6-axis ft sensors.
Links
The links are the physical rigid bodies that constitute the robot. Each link is characterised by a mass (represented in models by the inertial parameters) and a physical shape (represented in models by meshes).
For defining the links we are representing the robot as a directed tree, where the
root_link
is the body that can be attached to the pole, and the meaning of the other links can be deduced by their parent joint, as defined in the previous section.The main idea behind this naming scheme is that "big" links (roughly speaking, the one that can reasonably interact with the user) are named with intuitive names such as
upper_arm
,forearm
,upper_leg
,lower_leg
,chest
,head
,foot
,hand
. All the little links that instead are part of more complex linkages, take their name from the articulation, such astorso_1
,torso_2
.Frames
In literature and in robotics software, the links and frames concepts are often confused, because every link is usually associated with a frame rigidly attached to it. However this link frame definition is dependent on the formalism that one uses for describing the robot. For example, if the Denavit Hartenberg convention is used the link frame origin is required to be placed on the axis of the child joint, while if the Modified Denavit Hartenberg convention or the URDF format the link frame origin is required to be place on the axis of the parent joint.
To avoid inconsistency, I think it is better to clearly separate frame and link concepts.
For the iCub, it could be useful to explicitly state the frame used for defining the kinematic chains used in the iKin library.
In particular a useful set of frames could be:
Skin Frames
An interesting set of frame is the frame defined by the iKin convention ( http://eris.liralab.it/wiki/ICubForwardKinematics ). This reference frames are used by the skin system to express contact points, force and torques (in skinDynLib data structures) and taxel positions.
The one currently used by the skin system are:
LEFT_FOOT
SkinPartRIGHT_FOOT
SkinPartSKIN_LEFT_UPPER_ARM
SkinPartSKIN_LEFT_FOREARM
SkinPartSKIN_LEFT_HAND
SkinPartSKIN_RIGHT_UPPER_ARM
SkinPartSKIN_RIGHT_FOREARM
SkinPartSKIN_RIGHT_HAND
SkinPartSkinPart
RIGHT_LEG_UPPER
,LEFT_LEG_UPPER
,RIGHT_LEG_LOWER
,LEFT_LEG_LOWER
andSKIN_FRONT_TORSO
are not currently correctly processed by the system and they are not assigned any frame. In particularSKIN_FRONT_TORSO
is problematic because the Torso link for the DH convention used by iKin for frame placement has three different frames, depending on which chain is considered (if head, right arm or left arm).For all this frames, an alternative naming could be:
"linkName_ikin_frame", to have a name that is directly related to the iKin software.
Other frames
Probably some other intermediate frame could also be useful, as something related to the neck.
Additional frames could be defined to improve compliance with REP-120 that is already used for different humanoids URDFs (Nao, Romeo, HRP-2, Coman).
This is an initial proposal that is not discussing several important aspects (eyes, hands, accelerometers, gyroscopes), but I guess it is already ready for getting some feedback.
cc @EnricoMingo @jeljaik @arocchi @hu-yue
The text was updated successfully, but these errors were encountered: