Closed rgoulter closed 6 years ago
Thoughts/concerns that the code can fail-an-assert?
I guess I'd hoped that using double
instead of LongInt
would make it much faster.
It would be 'wrong' to "break" the GUI like this. -- It would be 'acceptable' if the quick / broken one were a choice.
In any case, I think this PR is loosely an improvement. And that the project has other issues besides speed. -- Probably offering which 'algorithm' to use would be a good idea. (And probably implementing Fortune's would be a better idea. -- Or making the Delaunay one ... visual / informative).
Some notes:
signDet
method used by the "point in circle" method of Delaunay PointSet.int
isn't viable as an imprecise alternative to our preciseLongInt
.double
... passes the unit tests. But it seems like there are assumptions which break.