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

[50$ Bounty] PolyBool can cause an editor crash on T-Junctions #17

Open
Henry00IS opened this issue Sep 14, 2022 · 0 comments
Open

[50$ Bounty] PolyBool can cause an editor crash on T-Junctions #17

Henry00IS opened this issue Sep 14, 2022 · 0 comments
Labels
bounty You will get paid if you do this bug Something isn't working

Comments

@Henry00IS
Copy link
Owner

Two shapes that look like a B

Boolean operations mistakenly consider these two shapes as a single shape with an infinitely thin point connecting them, causing the convex decomposition to fail. This will then cause a stack overflow and editor crash in the BayazitDecomposer. The splitting of this type of polygon at the singularity has yet to be fixed.

It's likely that PolyBoolJS (and thus PolyBoolCS) has this as an oversight in their algorithm. They scan lines vertically and end up merging them together. Maybe this is correct in their eyes. But we need a way to detect and split these singularities into multiple shapes. Whether you fix it here as a post-process or there in the core algorithm, it would be an incredible achievement and I will be super thankful!

Example Projects: S2D Projects Bad.zip

The payment of 50$ (or 50€ if preferred) will be made via PayPal after verifying your work.

@Henry00IS Henry00IS added bug Something isn't working bounty You will get paid if you do this labels Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bounty You will get paid if you do this bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant