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

Documentation for computeAABB() #40

Open
mkoval opened this issue Aug 4, 2014 · 0 comments
Open

Documentation for computeAABB() #40

mkoval opened this issue Aug 4, 2014 · 0 comments

Comments

@mkoval
Copy link

mkoval commented Aug 4, 2014

I spent a few hours debugging poor performance of FCL on some mesh models and realized that the broad-phase check wasn't pruning any pairs of geometries. I eventually tracked the problem down to incorrect AABBs on my CollisionObjects. It turns out that I wasn't calling computeAABB after changing the pose of my geometries, so the broad-phase check considered them all to be overlapping at the origin.

It would be helpful if you could add a note about this in the README. The current documentation doesn't mention computeAABB at all and, thus, gives the impression that the CollisionManager takes care of computing everything necessary for the broad-phase check.

(The performance is great now that the broad-phase check works!)

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

2 participants