Title: NURBS failure when ClassifyEdge fails (was: Difference fail depending on compilation options (was: Difference fail with OpenMP)) · Issue #1619 · solvespace/solvespace · GitHub
Open Graph Title: NURBS failure when ClassifyEdge fails (was: Difference fail depending on compilation options (was: Difference fail with OpenMP)) · Issue #1619 · solvespace/solvespace
X Title: NURBS failure when ClassifyEdge fails (was: Difference fail depending on compilation options (was: Difference fail with OpenMP)) · Issue #1619 · solvespace/solvespace
Description: This was first reported on the forum using the attached file. Issue_OpenMP.zip You can make the problem go away by: Dragging the original extrusion the other direction Using a version of Solvespace built without OpenMP Open it using the ...
Open Graph Description: This was first reported on the forum using the attached file. Issue_OpenMP.zip You can make the problem go away by: Dragging the original extrusion the other direction Using a version of Solvespace...
X Description: This was first reported on the forum using the attached file. Issue_OpenMP.zip You can make the problem go away by: Dragging the original extrusion the other direction Using a version of Solvespace...
Opengraph URL: https://github.com/solvespace/solvespace/issues/1619
X: @github
Domain: patch-diff.githubusercontent.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"NURBS failure when ClassifyEdge fails (was: Difference fail depending on compilation options (was: Difference fail with OpenMP))","articleBody":"This was first [reported on the forum](https://solvespace.com/forum.pl?action=viewthread\u0026parent=6129\u0026tt=1755262509) using the attached file.\n\n[Issue_OpenMP.zip](https://github.com/user-attachments/files/21822164/Issue_OpenMP.zip)\n\nYou can make the problem go away by:\n1) Dragging the original extrusion the other direction\n2) Using a version of Solvespace built without OpenMP\n3) Open it using the web version (which does not use OpenMP)\n\nTreating it as a concurrency problem I have been able to make the issue go away by commenting out the following line in SShell::CopySurfacesTrimAgainst()\n```\n#pragma omp parallel for\n```\nThat function calls MakeCopyTrimAgainst() and inside of that we can \"fix\" things by adding #pragma omp critical to either of the two big loops [1 here](https://github.com/solvespace/solvespace/blob/74a13ef61c490a400eaa49c5335622d91ada12fc/src/srf/boolean.cpp#L603) or the one under it [2 here](https://github.com/solvespace/solvespace/blob/74a13ef61c490a400eaa49c5335622d91ada12fc/src/srf/boolean.cpp#L637).\n\nI don't understand how it can be fixed by making a critical section for either of two loops. At this point I don't have any idea what's causing it.","author":{"url":"https://github.com/phkahler","@type":"Person","name":"phkahler"},"datePublished":"2025-08-17T12:37:19.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":17},"url":"https://github.com/1619/solvespace/issues/1619"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:2a51ca9e-77a3-20ac-883e-9c0c2d13607d |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | 994E:1E201A:63B86C:883CC3:696E7960 |
| html-safe-nonce | 1327ed75f9e401391ecbcf09608a3c80f07570e410aa93c64134a9971773d290 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5OTRFOjFFMjAxQTo2M0I4NkM6ODgzQ0MzOjY5NkU3OTYwIiwidmlzaXRvcl9pZCI6IjU2NTg4Mjg2ODIyMzUzNzgwMTYiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 899f55bdd6ea049086cd6935a82a084a8647dbb201916e08b8cf254c7775f0c5 |
| hovercard-subject-tag | issue:3328296115 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/solvespace/solvespace/1619/issue_layout |
| twitter:image | https://opengraph.githubassets.com/ab5a673550d98fd7fadb9a22d6a55b5edb1f51373b10a15203a9eceb26a7af55/solvespace/solvespace/issues/1619 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/ab5a673550d98fd7fadb9a22d6a55b5edb1f51373b10a15203a9eceb26a7af55/solvespace/solvespace/issues/1619 |
| og:image:alt | This was first reported on the forum using the attached file. Issue_OpenMP.zip You can make the problem go away by: Dragging the original extrusion the other direction Using a version of Solvespace... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | phkahler |
| hostname | github.com |
| expected-hostname | github.com |
| None | fdad15fd2ad43212aa8b8be5f2c2725550f8374ceeeb154a999ad9145b43f3f7 |
| turbo-cache-control | no-preview |
| go-import | github.com/solvespace/solvespace git https://github.com/solvespace/solvespace.git |
| octolytics-dimension-user_id | 18541596 |
| octolytics-dimension-user_login | solvespace |
| octolytics-dimension-repository_id | 32905910 |
| octolytics-dimension-repository_nwo | solvespace/solvespace |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 32905910 |
| octolytics-dimension-repository_network_root_nwo | solvespace/solvespace |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 27b23bc056eb973d350fc95afc848757edb9e7a9 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width