Skip to content

Add first_intersection / first_intersected_primitive to CGAL_AABB_tree bindings #186

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

Open
noamgat opened this issue Oct 25, 2020 · 1 comment · May be fixed by #187
Open

Add first_intersection / first_intersected_primitive to CGAL_AABB_tree bindings #186

noamgat opened this issue Oct 25, 2020 · 1 comment · May be fixed by #187

Comments

@noamgat
Copy link

noamgat commented Oct 25, 2020

Just like any_intersection() / any_intersected_primitive() have been added to the swig binding for CGAL_AABB_tree, it would be very helpful if first_intersection / first_intersected_primitive() would be added as well (even without the skip functor), as they are the most important data when raycasting. It should be easy, given that the any variant is there.

If that is impossible, perhaps updating the python example AABB_triangle_3_example.py for using "all_intersected_primitives"? it expects a "Primitive iterator helper" which I have no idea how to construct in python.

Thanks!

@noamgat
Copy link
Author

noamgat commented Oct 26, 2020

Created a PR: #187

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

Successfully merging a pull request may close this issue.

1 participant