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

CompoundMesh does not work with FCL #1053

Open
rjoomen opened this issue Oct 21, 2024 · 6 comments
Open

CompoundMesh does not work with FCL #1053

rjoomen opened this issue Oct 21, 2024 · 6 comments
Assignees

Comments

@rjoomen
Copy link
Contributor

rjoomen commented Oct 21, 2024

CompoundMesh does not work with the FCL collision checker. I suppose it is because for Bullet, one CompoundMesh is added as one compound object, whereas for FCL, one CompoundMesh is added a multiple objects.

I've taken a look, it seems FCL does not support compound objects, apart from BHVModel, which only supports meshes or pointclouds as submodels.

And for reference, neither does HPP-FCL: #340, #448.

@Levi-Armstrong
Copy link
Contributor

Are you saying there is a bug? The concept of CompoundMesh has always existed just handled differently now. Previously every sub mesh was converted directly to a Geometry object but now all sub meshes are stored in the new CompoundMesh. Then the collision checker handle this new type and processes all of the sub meshes. The only difference now is the CompoundMesh is captured at the highest level.

@Levi-Armstrong
Copy link
Contributor

Here is the code which process the CompoundMesh.

@rjoomen
Copy link
Contributor Author

rjoomen commented Oct 21, 2024

I checked the code indeed. There is a bug, I tested adding some meshes separately (within a link) or as a CompoundMesh, and for Bullet this works for both, but for FCL the CompoundMesh version crashes. And with my HPP-FCL version (mostly similar to the FCL version) it is the same. I did not check in detail what went wrong, but CompoundMesh is the cause.

@Levi-Armstrong
Copy link
Contributor

I thought this was covered by a unit test but I will look into it.

@Levi-Armstrong
Copy link
Contributor

There is a unit test which does confirm this is working but may not be testing everything. Do you have a backtrace or could you add a unit test which triggers the failure?

@rjoomen
Copy link
Contributor Author

rjoomen commented Oct 27, 2024

I'll take a look tomorrow. I'm not really using FCL, so I just found this by accident.

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

2 participants