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

Intersection semantics #27

Open
Rhuagh opened this issue Sep 10, 2017 · 4 comments
Open

Intersection semantics #27

Rhuagh opened this issue Sep 10, 2017 · 4 comments

Comments

@Rhuagh
Copy link
Collaborator

Rhuagh commented Sep 10, 2017

We should define what a border hit should do for all intersection tests.

@kvark
Copy link
Collaborator

kvark commented Sep 11, 2017

One action item to take - see what other libraries are doing.
In general, I'm fine with either decision as long as it's consistent ;)

@Rhuagh
Copy link
Collaborator Author

Rhuagh commented Sep 11, 2017

Yeah, I tried finding information on this, but my google skills apparently aren't up to the task, and I'm not sure what open libraries there are except for ncollide.

@Rhuagh
Copy link
Collaborator Author

Rhuagh commented Sep 11, 2017

ncollide seems to report a hit on border intersections. It's hard to tell ...

@msiglreith
Copy link

Related point would be checking the returned values on edge cases. ncollide for example had issues at 0 penetration depth, resulting in bogus normals returned from the hit test.

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

3 participants