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

FCL Integration Status #7656

Closed
5 of 9 tasks
avalenzu opened this issue Dec 21, 2017 · 3 comments
Closed
5 of 9 tasks

FCL Integration Status #7656

avalenzu opened this issue Dec 21, 2017 · 3 comments
Assignees

Comments

@avalenzu
Copy link
Contributor

avalenzu commented Dec 21, 2017

This issue serves to track the status of our efforts to use FCL as the primary (only?) collision engine in Drake. The checklist below should be updated as tasks are completed.

Upstream fixes for working with boxes

(Drake branch)

  • There are multiple issues in the FCL Box-Box- collision:
    • Signed values are reported as negative instead of as positive. (Addressed by fcl PR 259.)
    • The reported "contact position" lies on the surface of one of the boxes instead of midway between the two deepest penetrating points (it is not clear that which box it lies on is always the first or second; it may depend on configuration). (Addressed by fcl PR 259.)
    • Collision between Box and quadratic surfaces (e.g., Sphere and Cylinder) produces highly imprecise solutions. See proximity_engine_test.cc -- under the BoxPenetrationTest tests for concrete examples.

Upstream fixes for working with capsules

(FCL branch | Drake branch)

Upstream fixes for working with convex polyhedra

(libccd branch | FCL branch | Drake branch)

Issues with cylinders

There may be additional issues that we should consider in the discussion on ipab-slmc/exotica#184.

@amcastro-tri
Copy link
Contributor

@avalenzu. I was thinking about spiking an MBP/GS demo while at HQ in two weeks. Is the mesh (convex, just convex) penetration depth query working with FCL? would you have any unit test/example that shows failure modes?

@sherm1
Copy link
Member

sherm1 commented May 11, 2020

@SeanCurtis-TRI is this worth keeping around?

@SeanCurtis-TRI
Copy link
Contributor

Yeah. We can close it. Of the remaining open issues, only the putative error in MRP collision point computation is of interest to me. The fix doesn't come with a test so it'll have to be investigated from scratch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants