Skip to content

Comments

Boolean fuse operation fails when rotation is involved.#1042

Draft
skelliam wants to merge 2 commits intoOpen-Cascade-SAS:masterfrom
skelliam:polar_fuse_test
Draft

Boolean fuse operation fails when rotation is involved.#1042
skelliam wants to merge 2 commits intoOpen-Cascade-SAS:masterfrom
skelliam:polar_fuse_test

Conversation

@skelliam
Copy link

@skelliam skelliam commented Feb 1, 2026

DRAFT

Here are some test cases that demonstrate issue #1041.

Note: I have signed and uploaded the CLA today, 1-Feb-2026.

image

@skelliam
Copy link
Author

skelliam commented Feb 1, 2026

interesting that there are different failures across different platforms... suggesting maybe some kind of floating point issue?

Windows failures:

  • ThreeOccurrences120Degrees
  • FiveOccurrences72Degrees
  • SixOccurrences60Degrees

Mac with clang failures:

  • FiveOccurrences72Degrees
  • SixOccurrences60Degrees

Linux with clang failures:

  • ThreeOccurrences120Degrees
  • FiveOccurrences72Degrees

@dpasukhi dpasukhi requested review from dpasukhi and removed request for dpasukhi February 1, 2026 23:16
@dpasukhi
Copy link
Member

dpasukhi commented Feb 1, 2026

Yes, the impact is very interesting. I will check it in coming days, can be issue with some signal handler for example (I will check on each of os locally)

@skelliam
Copy link
Author

skelliam commented Feb 2, 2026

Yes, the impact is very interesting. I will check it in coming days, can be issue with some signal handler for example (I will check on each of os locally)

Great, thank you. I have no idea where to start. I spent some hours today with Claude and didn't really get anywhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants