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

Switch CGAL to use Exact_predicates_inexact_constructions_kernel #17

Open
aabtop opened this issue Jun 9, 2020 · 1 comment
Open

Switch CGAL to use Exact_predicates_inexact_constructions_kernel #17

aabtop opened this issue Jun 9, 2020 · 1 comment

Comments

@aabtop
Copy link
Owner

aabtop commented Jun 9, 2020

Currently we are using the most accurate type of Kernel, Exact_predicates_exact_constructions_kernel, however it is also the slowest.

Most third party code examples using CGAL use Exact_predicates_inexact_constructions_kernel, so it is likely noticeably faster, however when I try switching there's a few compile errors to work though. It's probably worth working through them though.

@aabtop
Copy link
Owner Author

aabtop commented Jun 9, 2020

Note that the DooSabin subdivision is left out of the Subdivide method because it fails to compile, and I suspect it might be because of the use of Exact_predicates_exact_constructions_kernel.

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