Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

MeshShapeCollisionTraversalNode::leafCollides can create a contact with all values set to NaN #636

Open
jorisv opened this issue Dec 6, 2024 · 2 comments

Comments

@jorisv
Copy link
Contributor

jorisv commented Dec 6, 2024

When running the test test_talos in pinocchio-test-cpp-parallel-geometry we run into a failure because geometry_data_ref.collisionResults[k].getContacts() is filled with NaN.

This is because runGJKAndEPA fall in the details::GJK::NoCollisionEarlyStopped stop criterion. Then, MeshShapeCollisionTraversalNode::leafCollides don't filter this output and create a Contact even if there is no collision.

@lmontaut
Copy link
Contributor

lmontaut commented Dec 6, 2024

Thanks @jorisv
I will look into it soon

@sea-bass
Copy link

I'm also seeing this with the Python bindings in trying to upgrade my library. Specifically:

  1. Get this branch: Upgrade to Pinocchio 3.3.1 and Drake 1.36.0 sea-bass/pyroboplan#77
  2. Try out the examples/collision_along_path.py example
  3. The collisions between the Franka meshes and the cube are identified, but all the nearest points are NaN. The sphere collisions look fine.

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

No branches or pull requests

3 participants