-
Notifications
You must be signed in to change notification settings - Fork 208
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
straight lines issue #15
Comments
Please be more specific. What did you expect, and what do you get instead? Can you set up a test case with exact coordinates and tolerance values used? |
Here you go: http://s3.postimg.org/67v1cbezn/example.png As you can see, I thought it would respect the corner points, and give me only the corners with a very high tolerance. Coordinates, outline: |
That's pretty weird, shouldn't work this way. I'll check this out. |
mmm maybe its my code then, check this later
|
@gcmartijn yeah, looking at the picture, it looks like it shows everything except the points that simplify.js retained. |
I'm now using your code directly to check where the problem is. If you run it, you can see the wrong dots. |
@gcmartijn hmm, your fiddle doesn't show anything for me in Chrome, just a blank screen. |
@gcmartijn nevermind, seeing the points. If you run it with proper tolerance (say 1), the result is good — just the important corners. |
I'm using the latest chrome, don't you see the red pixel (right side) when when push Run? |
I see now, maybe its the python module, going to recheck that again. |
I convert your code to a language I'm using.
But it's not possible to get only the 'important' points if needed.
Maybe I can't use your code, maybe I can, I don't know.
Look at this image: http://s16.postimg.org/afccosymd/test_col.png
I have the exact outline coordinate's, what I need is only the corners (in this case)
So 16 x,y points.
When I set the tolerance using your script very high, it will create gaps inside the corners...
The text was updated successfully, but these errors were encountered: