robotology / icub-models-generator

Resources and programs to generated models (URDF, SDF) of the iCub robot
14 stars 23 forks source link

wrong joint limits in generated files #5

Open andreadelprete opened 10 years ago

andreadelprete commented 10 years ago

It seems that the joint limits for the two hip-roll joints are wrong. See for instance the left-hip-roll joint of generated/gazebo_models/iCubGenova03/iCubGenova03.sdf:

<joint name='l_hip_roll' type='revolute'>
      <child>l_hip_2</child>
      <parent>l_hip_1</parent>
      <axis>
        <xyz>-1 2.65359e-06 7.34641e-06</xyz>
        <limit>
          <lower>-2.07694</lower>
          <upper>0.296706</upper>
          <effort>84</effort>
          <velocity>100</velocity>
        </limit>
        <dynamics>
          <damping>0.1</damping>
        </dynamics>
        <use_parent_model_frame>1</use_parent_model_frame>
      </axis>
    </joint>

and compare the joint limits with the one of the model of the model /icub/icub.sdf in the icub_gazebo repository:

        <joint name='l_hip_roll' type='revolute'>
            <child>l_hip_2</child>
            <parent>l_hip_1</parent>
            <axis>
                <xyz>-1 5.30716e-06 1.26536e-05</xyz>
                <limit>
                    <lower>-0.34</lower>
                    <upper>1.6707963268</upper>
                    <effort>10000</effort>
                    <velocity>100</velocity>
                </limit>
                <dynamics>
                    <damping>0.1</damping>
                </dynamics>
            </axis>
        </joint>

The problem is the same for the right-hip roll and for the iCubGenova01 model.

andreadelprete commented 10 years ago

Also the ankle-pitch limits are significantly different. For the left-ankle pitch icub.sdf has:

<lower>-0.4490658504</lower>
<upper>0.8679448709</upper>

whereas iCubGenova03.sdf has:

          <lower>-0.733038</lower>
          <upper>0.366519</upper>
traversaro commented 9 years ago

cc @luca-fiorio @francesco-romano @jeljaik

jeljaik commented 9 years ago

Yes, we noticed just yesterday.

Jorhabib Eljaik Gómez,Ph.D. Student - Istituto Italiano di Tecnologia Dept. of Robotics, Brain and Cognitive Sciences Cognitive Humanoids Laboratory e-mail: jorhabib.eljaik@iit.it Cel: +39 388 352 33 85 Tel: +39 010 71781 420 Fax: +39 010 7170817

On Wed, Dec 10, 2014 at 10:20 AM, Silvio Traversaro < notifications@github.com> wrote:

cc @luca-fiorio https://github.com/luca-fiorio @francesco-romano https://github.com/francesco-romano @jeljaik https://github.com/jeljaik

— Reply to this email directly or view it on GitHub https://github.com/robotology-playground/icub-model-generator/issues/5#issuecomment-66423814 .

iron76 commented 9 years ago

In my opinion it is not an easy task to define span [j_min, j_max] for all joints. It might be feasible only for some. There are situations in which the range of a joint depends on previous and therefore the configuration space is quite a complex manifold. If we really want something like [j_min, j_max], we can try to list some possibilities:

traversaro commented 8 years ago

Ref: https://github.com/robotology-playground/icub-gazebo/pull/27 .

traversaro commented 8 years ago

This is a general instance of the problem of getting correct joint limits.

traversaro commented 7 years ago

cc @S-Dafarra