Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Checking on a potential bug #933

Closed
wants to merge 4 commits into from
Closed

Checking on a potential bug #933

wants to merge 4 commits into from

Conversation

elalish
Copy link
Owner

@elalish elalish commented Sep 17, 2024

This test passes for me locally on Mac, so checking the CI. Found here.

@elalish elalish self-assigned this Sep 17, 2024
@pca006132
Copy link
Collaborator

Hmm, I'm feeling that this is related to compiler optimization...

@pca006132
Copy link
Collaborator

ok, we didn't really run the boolean lol

@elalish
Copy link
Owner Author

elalish commented Sep 17, 2024

Oh, duh, thanks!

@elalish
Copy link
Owner Author

elalish commented Sep 17, 2024

Okay, and just like that - I have a local repro. Just another example of why every piece of pasted code should just be PR instead. 😄

@pca006132
Copy link
Collaborator

My original snippet doesn't have if (options.exportModels) :P

@elalish
Copy link
Owner Author

elalish commented Sep 17, 2024

This one's not a valid test - the first mesh is strongly self-intersecting, hence why we get self-intersecting polygons that make inverted triangles.

Screen.Recording.2024-09-17.at.9.27.07.AM.mov

@elalish elalish closed this Sep 17, 2024
@elalish elalish deleted the boolBug branch September 17, 2024 16:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants