moveit / moveit_resources

URDFs, meshes, and config packages for MoveIt testing
61 stars 114 forks source link

[panda] joint limits do not match the specification #116

Open squizz617 opened 2 years ago

squizz617 commented 2 years ago

Hi.

The joint limits defined in the URDF of the panda manipulator are either (1) different from the specification or (2) slightly exceed the specified min/max.

Expected (as per the datasheet provided by the manufacturer):

Defined soft lower and upper limits:

Could you consider fixing these? Thank you.

rhaschke commented 2 years ago

Thanks for the hint! Could you please file a pull request to fix those? Note that this is not the official URDF of the Panda robot, but one that is used internally in MoveIt CI. For the official version, see https://github.com/frankaemika/franka_ros/tree/noetic-devel/franka_description

rhaschke commented 2 years ago

@squizz617, do you plan to submit a PR to fix this? If not, I'm tempted to close this.

squizz617 commented 2 years ago

Hi @rhaschke, I've opened #118 for this. Thanks!

BTW,

this is not the official URDF of the Panda robot, but one that is used internally in MoveIt CI.

this URDF is being used in the tutorials, where it mentions that "the Franka Emika Panda robot is used". So I assumed this file was the official one.