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

Improve gazebo hand/finger collision behavoir #147

Merged

Conversation

rickstaa
Copy link
Contributor

@rickstaa rickstaa commented Aug 13, 2021

This pull request improves the Gazebo collision geometries.

Old collision geometries

image
image

New collision geometries

image
image

@rickstaa rickstaa force-pushed the improve_gazebo_hand_collision_geometries branch from 8d099f9 to 0ac476c Compare August 13, 2021 09:00
@rickstaa rickstaa changed the title improve gazebo hand collision geometries Improve gazebo hand/finger collision behavoir Aug 13, 2021
@rickstaa rickstaa force-pushed the improve_gazebo_hand_collision_geometries branch 4 times, most recently from 0668131 to 29f08f4 Compare August 17, 2021 13:21
@rickstaa rickstaa changed the base branch from noetic-devel to develop September 28, 2021 08:28
@rickstaa rickstaa force-pushed the improve_gazebo_hand_collision_geometries branch from 29f08f4 to e2e3bfd Compare September 28, 2021 08:37
@falfab
Copy link
Contributor

falfab commented Oct 12, 2021

Hi @rickstaa
thank you for your contribute! I have tested the new fingers and hand collision meshes and they improve the collision model of Gazebo.

If you could add the CHANGELOG entry this PR will be ready to be merged.

…ption

This commit replaces the hand/finger collision geometries that are used in the Gazebo
robot description with more accurate meshes. These meshes were taken from
[AndrejOrsula/panda_ign](https://github.com/AndrejOrsula/panda_ign/blob/master/panda/meshes/collision/finger.stl).
@rickstaa rickstaa force-pushed the improve_gazebo_hand_collision_geometries branch from e2e3bfd to 7327078 Compare October 12, 2021 16:24
@rickstaa
Copy link
Contributor Author

rickstaa commented Oct 12, 2021

@falfab Thanks for accepting my pull request! I applied the changes you requested. I also merged the two commits into one clean commit. Feel free to request any other changes.

@falfab falfab merged commit 38fa8a9 into frankaemika:develop Oct 13, 2021
rickstaa added a commit to rickstaa/franka_ros that referenced this pull request Oct 18, 2021
This commit fixes a bug that was introduced by frankaemika#147. In frankaemika#147 the
finger collision geometries were replaced by more accurate meshes.
These more accurate messages however broke the 'gripper_action' action.
This commit migrates back to using geomteries for the finger collision goeometries.
The action server works fine when geometries are used.
rickstaa added a commit to rickstaa/franka_ros that referenced this pull request Oct 18, 2021
This commit fixes a bug that was introduced by frankaemika#147. In frankaemika#147 the
finger collision geometries were replaced by more accurate meshes.
These more accurate messages however broke the 'gripper_action' action.
This commit migrates back to using geomteries for the finger collision goeometries.
The action server works fine when geometries are used.
rickstaa added a commit to rickstaa/franka_ros that referenced this pull request Oct 18, 2021
This commit fixes a bug that was introduced by frankaemika#147. In frankaemika#147 the
finger collision geometries were replaced by more accurate meshes.
These more accurate messages however broke the 'gripper_action' action.
This commit migrates back to using geometries for the finger collision geometries.
The action server works fine when geometriesare used.
rickstaa added a commit to rickstaa/franka_ros that referenced this pull request Oct 18, 2021
This commit fixes a bug that was introduced by frankaemika#147. In frankaemika#147 the
finger collision geometries were replaced by more accurate meshes.
These more accurate messages however broke the 'gripper_action' action.
This commit migrates back to using geometries for the finger collision geometries.
The action server works fine when geometriesare used.
rickstaa added a commit to rickstaa/franka_ros that referenced this pull request Oct 20, 2021
This commit fixes a bug that was introduced by frankaemika#147. In frankaemika#147 the
finger collision geometries were replaced by more accurate meshes.
These more accurate messages however broke the 'gripper_action' action.
This commit migrates back to using geometries for the finger collision geometries.
The action server works fine when geometriesare used.
rickstaa added a commit to rickstaa/franka_ros that referenced this pull request Oct 20, 2021
This commit fixes a bug that was introduced by frankaemika#147. In frankaemika#147 the
finger collision geometries were replaced by more accurate meshes.
These more accurate messages however broke the 'gripper_action' action.
This commit migrates back to using geometries for the finger collision geometries.
The action server works fine when geometriesare used.
m-elwin pushed a commit to m-elwin/franka_ros that referenced this pull request Oct 25, 2021
This commit fixes a bug that was introduced by frankaemika#147. In frankaemika#147 the
finger collision geometries were replaced by more accurate meshes.
These more accurate messages however broke the 'gripper_action' action.
This commit migrates back to using geometries for the finger collision geometries.
The action server works fine when geometriesare used.
rickstaa added a commit to rickstaa/franka_ros that referenced this pull request Oct 28, 2021
This commit fixes a bug that was introduced by frankaemika#147. In frankaemika#147 the
finger collision geometries were replaced by more accurate meshes.
These more accurate messages however broke the 'gripper_action' action.
This commit migrates back to using geometries for the finger collision geometries.
The action server works fine when geometriesare used.
falfab added a commit that referenced this pull request Nov 5, 2021
…kins-script-must-fail-when-internal to develop

* commit 'f8506671ea911c35f6c5794d8d8a7c326f903f20':
  Modified checkgithistory to accept parameters and fixed typos in comment
  Checking github history sync in separate jenkins stage and removed unused dependency from dockerfile
  Add: bash script to check sync between local and public repos
@rickstaa rickstaa deleted the improve_gazebo_hand_collision_geometries branch November 9, 2021 14:47
tahsinkose pushed a commit to tahsinkose/franka_ros that referenced this pull request Nov 11, 2021
This commit fixes a bug that was introduced by frankaemika#147. In frankaemika#147 the
finger collision geometries were replaced by more accurate meshes.
These more accurate messages however broke the 'gripper_action' action.
This commit migrates back to using geometries for the finger collision geometries.
The action server works fine when geometriesare used.
rickstaa added a commit to rickstaa/franka_ros that referenced this pull request Nov 22, 2021
This commit fixes a bug that was introduced by frankaemika#147. In frankaemika#147 the
finger collision geometries were replaced by more accurate meshes.
These more accurate messages however broke the 'gripper_action' action.
This commit migrates back to using geometries for the finger collision geometries.
The action server works fine when geometriesare used.
rickstaa added a commit to rickstaa/franka_ros that referenced this pull request Nov 23, 2021
This commit fixes a bug that was introduced by frankaemika#147. In frankaemika#147 the
finger collision geometries were replaced by more accurate meshes.
These more accurate messages however broke the 'gripper_action' action.
This commit migrates back to using geometries for the finger collision geometries.
The action server works fine when geometriesare used.
rickstaa added a commit to rickstaa/franka_ros that referenced this pull request Nov 23, 2021
This commit fixes a bug that was introduced by frankaemika#147. In frankaemika#147 the
finger collision geometries were replaced by more accurate meshes.
These more accurate messages however broke the 'gripper_action' action.
This commit migrates back to using geometries for the finger collision geometries.
The action server works fine when geometriesare used.
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.

2 participants