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

Visibility bug corner case? #12

Open
azrafe7 opened this issue Jun 8, 2016 · 2 comments
Open

Visibility bug corner case? #12

azrafe7 opened this issue Jun 8, 2016 · 2 comments

Comments

@azrafe7
Copy link
Owner

azrafe7 commented Jun 8, 2016

This polygon is not correctly decomposed by SnoeyinkKeil (it's a particular of big.png test image):

big_particular

decomp:
vis_issue

poly points:

poly = [32,3, 18,9, 16,9, 17,6, 14,8, 9,4, 9,9, 0,12, 0,15, 12,12, 10,21, 20,15, 29,17, 21,12, 21,10, 32,7, 32,4 /*, 32,3*/];

It could also affect Bayazit, since now it's using Visibility.

Works fine with the original java implementation from Snoeyink it seems, so the problem must be on my side.

@azrafe7 azrafe7 changed the title Visibility bug corner case Visibility bug corner case? Jun 8, 2016
@azrafe7
Copy link
Owner Author

azrafe7 commented Jun 8, 2016

Note to self: check popVisible().

@azrafe7
Copy link
Owner Author

azrafe7 commented Jul 7, 2016

To anyone reading the issue: if you encounter a similar problem please post your findings here, as I'd really like to narrow it down and fix it!

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