Skip to content
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

[feature_request] Move hand collision geometries to the hand joints #163

Open
rickstaa opened this issue Aug 30, 2021 · 1 comment
Open

Comments

@rickstaa
Copy link
Contributor

rickstaa commented Aug 30, 2021

In the current robot description (i.e. franka_ros v0.8.0) the collision geometries for the fingers are found inside the hand joint.

<collision>
<origin xyz="0 0.05 0.04" rpy="0 0 0"/>
<geometry>
<sphere radius="${0.04+safety_distance}" />
</geometry>
</collision>
<collision>
<origin xyz="0 0 0.1" rpy="0 ${pi/2} ${pi/2}"/>
<geometry>
<cylinder radius="${0.02+safety_distance}" length="0.1" />
</geometry>
</collision>
<collision>
<origin xyz="0 -0.05 0.1" rpy="0 0 0"/>
<geometry>
<sphere radius="${0.02+safety_distance}" />
</geometry>
</collision>
<collision>
<origin xyz="0 0.05 0.1" rpy="0 0 0"/>
<geometry>
<sphere radius="${0.02+safety_distance}" />
</geometry>
</collision>

Unfortunately, third party packages like MoveIt expect each controllable joint to have a collision mesh/geometry. If these are not found, the following warnings are thrown:

[ WARN] [1628672207.286446541]: Link panda_leftfinger has visual geometry but no collision geometry. Collision geometry will be left empty. Fix your URDF file by explicitly specifying collision geometry.
[ WARN] [1628672207.286477598]: Link panda_rightfinger has visual geometry but no collision geometry. Collision geometry will be left empty. Fix your URDF file by explicitly specifying collision geometry.

This can easily be solved by moving the collision geometries from the hand joint to the finger joint. I created three possible solutions to this problem (i.e. #151, #150 or #152).

Steps to reproduce the issue

  1. Clone https://github.com/rickstaa/panda_moveit_config/tree/noetic-devel-update in a catkin workspace together with the latest version of franka_ros.
  2. Checkout the noetic-devel-update branch.
  3. Build the catkin workspace.
  4. Start the demo launch file roslaunch panda_moveit_config demo.launch.
  5. See the warnings.
@rickstaa
Copy link
Contributor Author

rickstaa commented Nov 8, 2021

Will be fixed when #188 is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant