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

Couple of things with classifySphere #17

Open
yonatan opened this issue Apr 13, 2012 · 0 comments
Open

Couple of things with classifySphere #17

yonatan opened this issue Apr 13, 2012 · 0 comments

Comments

@yonatan
Copy link

yonatan commented Apr 13, 2012

(and probably classifyRadius as well)

First, it returns INTERSECT immediately when it detects an intersection with any of the frustum planes, which seems wrong when the sphere is completely on the outside of some other frustum plane.

Second, there is a (literal) corner case, when a sphere intersects 3 of the frustum planes and is on the inside of the other 3, while still being completely outside the frustum volume. I have no idea how to fix that one.

Demo with a frustum extracted from the identity matrix (which I guess spans -1 and 1 on all axes):
http://wonderfl.net/c/eNu4
Click the buttons for values demonstrating both issues.

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