Skip to content

Update hand collision mesh due to the collision issues in Octomap and MoveIt #65

Open
oguzhanbzglu wants to merge 1 commit into
enactic:mainfrom
b-robotized-forks:update_hand_collision
Open

Update hand collision mesh due to the collision issues in Octomap and MoveIt #65
oguzhanbzglu wants to merge 1 commit into
enactic:mainfrom
b-robotized-forks:update_hand_collision

Conversation

@oguzhanbzglu

@oguzhanbzglu oguzhanbzglu commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Problem

This MR updates the hand collision mesh (STL) to address collision issues observed in ROS 2 MoveIt when using OctoMap-based environment perception.

We are running the bimanual v1.0 robot in Isaac Sim with a top-mounted camera. The camera captures both the gripper and part of the hand assembly:

image

OctoMap generates collision objects from the depth image and is expected to ignore geometry that belongs to the robot itself. However, as shown below, OctoMap incorrectly creates collision voxels around the metal section at the base of the gripper:

image image

The root cause is that the curved portion of the metal component is not fully represented in the current collision mesh. As a result, this geometry is not filtered out by the self-collision masking and is perceived as an external obstacle. The same issue could potentially lead to inaccurate collision checking on real hardware.

Solution

The hand collision mesh has been slightly extended based on the curved size to fully cover the missing curved section of the metal component.

This update resolves the OctoMap self-filtering issue while keeping the mesh lightweight. The resulting STL file size remains effectively unchanged.

image

Copilot AI review requested due to automatic review settings June 5, 2026 11:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kou

kou commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

@stevenaya Could you take a look at this?

@stevenaya

Copy link
Copy Markdown
Contributor

We will first merge #64, then see if this problem remains.

@stevenaya

stevenaya commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

I think we may consider updating the collision mesh for v1.0 later to provide more detailed collision handling. Let me discuss this with our team member.

@oguzhanbzglu

Copy link
Copy Markdown
Contributor Author

Hello, thank you for considering this change. However, even though this change is independent of #64, if you expand the collision area, you might also want to consider modifying the inertia properties of the link defined in #64 to account for the change in mass.

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.

4 participants