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

move finger collision geometry sparse #152

Conversation

rickstaa
Copy link
Contributor

@rickstaa rickstaa commented Aug 18, 2021

Similar to #151 but now the cylinder geometry between the two fingers have been removed.

New geometries

Safety distance 0.03

image

Safety distance 0.0

image

Old geometries

Safety distance 0.03

image

Safety distance 0.0

image

@rickstaa rickstaa force-pushed the move_finger_collision_geometry_sparse branch from 9682837 to 36f601f Compare August 18, 2021 14:26
@rickstaa rickstaa changed the base branch from noetic-devel to develop September 28, 2021 08:41
@rickstaa rickstaa force-pushed the move_finger_collision_geometry_sparse branch 3 times, most recently from 3d00a59 to 5eb3e00 Compare September 28, 2021 09:06
This commit moves the finger collision geometries to the finger joints.
This was done to prevent errors in third party libaries. Moveit for
example expects a collision geometry in each joint that can be
controlled.
@rickstaa rickstaa force-pushed the move_finger_collision_geometry_sparse branch from 5eb3e00 to a2ce1c7 Compare September 28, 2021 09:07
@rickstaa
Copy link
Contributor Author

rickstaa commented Nov 9, 2021

Closed since this is implemented in #189

@rickstaa rickstaa closed this Nov 9, 2021
@rickstaa rickstaa deleted the move_finger_collision_geometry_sparse branch November 9, 2021 14:46
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

Successfully merging this pull request may close these issues.

1 participant