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

Consider additional CCD configuration options #199

Open
RossNordby opened this issue Feb 4, 2022 · 0 comments
Open

Consider additional CCD configuration options #199

RossNordby opened this issue Feb 4, 2022 · 0 comments

Comments

@RossNordby
Copy link
Member

RossNordby commented Feb 4, 2022

NarrowPhase.AddBatchEntries currently avoids a sweep test if maximum estimated relative motion is less than the speculative margin. This is pretty reasonable and saves a lot of time.

There could be another feature here. Have one condition for choosing whether or not to execute a sweep, and another for the speculative margin actually used to generate contacts.

This would allow an 'early out' margin that is smaller than the final contact generation speculative margin. Potentially useful for cases where you want to be able to detect secondary collisions.

This would likely require expanding the options on ContinuousDetection, which in turn could cause some cache alignment considerations in CollidableDescription. Might be worth doing, maybe? If you did this, it may also be worth biting the bullet on the expanded size and including a cached linear velocity so that the narrow phase can use pair relative linear velocities to calculate a local effective speculative margin rather than just adding the contributing margins. (Testing showed trying to load velocities from the usual otherwise-untouched cachelines was a net loss.)

Pretty low value; very corner case.

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

1 participant