ConvexHull
ConvexHull copied to clipboard
Corrected a couple of errors
Hi,
When using the example code I discovered two details that I've corrected:
- The point sorting was being done in the incorrect order. (Y must be sorted after X).
- The last element of the hullPoints array should be removed in order to plot a correct convex hull.
I also added a simple .html example to play with, it shows two polylines, one with the old example and one with the new one.
Cheers! Jose
Thanks for the changes, Jose! I will take this opportunity to clean up the docs as well.
I have a question though, why did you rename the variable top to ttop in the js?
Hi Mauricio, thanks for the quick follow up. It was highlighted as a special keyword in vim. Since I'm no expert in JavaScript I thought that maybe that could rise some kind of error, but the name 'top' should work perfectly.
Thanks for the library, it was very useful!! Jose
Sent from a mobile device
On Jun 11, 2013, at 7:09 PM, Mauricio Gomes [email protected] wrote:
Thanks for the changes, Jose! I will take this opportunity to clean up the docs as well.
I have a question though, why did you rename the variable top to ttop in the js?
— Reply to this email directly or view it on GitHub.