Anonymous | Login | 2021-01-16 12:30 MSK | ![]() |
My View | View Issues | Change Log | Roadmap |
View Issue Details [ Jump to Notes ] [ Related Changesets ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0031835 | Community | [OCCT] OCCT:Modeling Algorithms | public | 2020-10-10 13:09 | 2020-12-02 17:13 | ||||
Reporter | Vico Liang | ||||||||
Assigned To | bugmaster | ||||||||
Priority | normal | Severity | block | ||||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | [OCCT] 7.5.0 | ||||||||
Target Version | [OCCT] 7.5.0 | Fixed in Version | [OCCT] 7.5.0 | ||||||
Summary | 0031835: Modeling Algorithms - step by step Boolean common produces bad shape on given three cylinders | ||||||||
Description | There are three cylinders with same size which are perpendicular with each other. 1. Load the given file three_cylinder.brep there are three cylinders: cylinder1, cylinder2, cylinder3. 2. do boolean common with any two shapes: common1 = cylinder1 x cylinder2 3. do bollean common between common1 and cylinder3. The final shape is bad from the result view. It maybe a modeling problem or meshing problem. Test environment is occt 7.5.0 beta. It works well in occt 7.4.0. | ||||||||
Steps To Reproduce | pload MODELING VISUALIZATION # OK #plane p1 0 0 -200 0 0 1 #plane p2 -200 0 0 1 0 0 #plane p3 0 -200 0 0 1 0 #pcylinder c_1 p1 100 400 #pcylinder c_2 p2 100 400 #pcylinder c_3 p3 100 400 # OK pcylinder c_1 100 400 ttranslate c_1 0 0 -200 pcylinder c_2 100 400 trotate c_2 0 0 0 1 0 0 90 ttranslate c_2 0 200 0 pcylinder c_3 100 400 trotate c_3 0 0 0 0 1 0 90 ttranslate c_3 -200 0 0 # KO #restore three_cylinder.brep c #explode c So # KO plane p1 -200 2.22044604925033e-14 12.6935294289015 1 -1.11022302462516e-16 0 pcylinder c_1 p1 100 400 plane p2 0 0 -187.306470571099 0 0 1 pcylinder c_2 p2 100 400 plane p3 0 200 12.6935294289015 0 -1 1.11022302462516e-16 pcylinder c_3 p3 100 400 bcommon r12 c_2 c_1 bcommon r123 r12 c_3 incmesh r123 0.1 vclear vinit View1 vdefaults -autotriang 0 vdisplay -dispMode 1 r123 | ||||||||
Tags | No tags attached. | ||||||||
Test case number | bugs/modalg_7/bug31835_1,bug31835_2 | ||||||||
Attached Files | ![]() ![]() ![]() ![]() ![]() ![]() | ||||||||
![]() |
|
(0095882) msv (developer) 2020-10-12 09:30 |
Dear Vico, I checked this test on OCCT 7.4.0. It works the same as in 7.5.0beta. Here is the draw script:restore three_cylinder.brep a explode a bcommon r1 a_1 a_2 puts [checkshape r1] bcommon r r1 a_3 puts [checkshape r] incmesh r 0.1 tricheck r Here is the output: This shape seems to be valid This shape seems to be valid cross face errors: {face1, node1, face2, node2, distance} {6 1 12 1 132.74946253594433} {7 1 12 1 106.25607663447413} {7 4 12 4 132.74946253594433} {10 1 12 1 106.25607663447413} free nodes (in pairs: face / node): {12 2} {12 3} {12 4} {12 5} {12 6} {12 7} {12 8} {12 9} {12 10} {12 11} {12 12} {12 13} {12 14} {12 15} {12 16} {12 17} {12 18} {12 19} {12 20} {12 21} {12 22} {12 23} {12 24} {12 25} {12 26} {12 27} {12 28} {12 29} {12 30} {12 31} {12 32} {12 33} {12 34} {12 35} {12 36} Free_links 0 Cross_face_errors 4 Async_edges 0 Free_nodes 35 Small triangles 0 So, it is not a regression issue. |
(0095883) msv (developer) 2020-10-12 09:36 |
The script gives no error in the version 7.3.0. However, the visual look of the result is incorrect, too. See the picture Clipboard01.png. |
(0095888) Vico Liang (developer) 2020-10-12 15:21 |
The visual look of the result is correct in version 7.4.0, but incorrect in 7.5.0 beta. anyway, this is a bug, right? |
(0095889) msv (developer) 2020-10-12 15:28 |
Yes, this is a bug. But on my computer (win64, msvc 2015) the bug is the same on 7.4.0. If it was a regression it would require its urgent fix before issuing the release 7.5.0. |
(0095890) msv (developer) 2020-10-12 15:32 |
I have attached the picture Clipboard02.png with the visual result on 7.4.0. |
(0095916) msv (developer) 2020-10-13 09:46 |
Dear Vico, on which platform it works well on 7.4.0? |
(0095923) Vico Liang (developer) 2020-10-13 11:10 |
Dear msv, I'm running on my desktop with msvc 2017. |
(0095926) msv (developer) 2020-10-13 12:20 |
Could you provide the extract of code that calls the operation? |
(0095930) git (administrator) 2020-10-13 14:30 |
Branch CR31835 has been created by emv. SHA-1: f7dbcf7512e2a3f4220b433913f2f96581219e1b Detailed log of new commits: Author: emv Date: Tue Oct 13 14:31:27 2020 +0300 0031835: Modeling Algorithms - step by step Boolean common produces bad shape on given three cylinders Fix inconsistency between IntAna_QuadQuadGeo(gp_Cylinder,gp_Cylinder) and AxeOperator::Coplanar(). |
(0095931) emv (developer) 2020-10-13 14:35 |
I've checked the result in OCCT 7.4.0 built with VS2017, it is similar to the one built with OCCT 7.5.0_beta. The case is not a regression. |
(0095933) git (administrator) 2020-10-13 14:59 |
Branch CR31835 has been updated forcibly by emv. SHA-1: c5779392f6d3138fa443695fd99cf4590b30b6a4 |
(0095940) Vico Liang (developer) 2020-10-13 17:30 |
Code snippet: BRepAlgoAPI_Common aCommonAlgoAPI; aCommonAlgoAPI.SetArguments(aArgumentShapes); aCommonAlgoAPI.SetTools(aToolShapes); aCommonAlgoAPI.SetFuzzyValue(1e-6); aCommonAlgoAPI.SetRunParallel(Standard_True); aCommonAlgoAPI.SetNonDestructive(Standard_True); aCommonAlgoAPI.SetToFillHistory(Standard_False); aCommonAlgoAPI.Build(); |
(0095945) git (administrator) 2020-10-13 21:35 |
Branch CR31835 has been updated forcibly by emv. SHA-1: df4f95f132b3192eef399285f93b97b4a2044915 |
(0095949) emv (developer) 2020-10-14 09:29 |
Dear Mikhail, could you please review the git branch CR31835? Test results - http://jenkins-test-12.nnov.opencascade.com/view/CR31835-master-emv/view/COMPARE/ [^] |
(0095954) msv (developer) 2020-10-14 10:48 |
For integration: occt - CR31835 products - none |
(0096033) bugmaster (administrator) 2020-10-17 13:22 |
Combination - OCCT branch : IR-2020-10-16 master SHA - ae9f4b64cacf0df612944b3694a3bdfa5f1f29cf a206de37fbfa0bf71bd534ae47192bbec23b8522 Products branch : IR-2020-10-16 SHA - fcb5abe005e152f7f923f4cf6c02acb07c027cdc was compiled on Linux, MacOS and Windows platforms and tested in optimize mode. Number of compiler warnings: No new/fixed warnings Regressions/Differences/Improvements: No regressions/differences CPU differences: Debian80-64: OCCT Total CPU difference: 18027.820000000127 / 18057.130000000117 [-0.16%] Products Total CPU difference: 12174.330000000093 / 12182.170000000115 [-0.06%] Windows-64-VC14: OCCT Total CPU difference: 19740.03125 / 19746.828125 [-0.03%] Products Total CPU difference: 13564.71875 / 13586.625 [-0.16%] Image differences : No differences that require special attention Memory differences : No differences that require special attention |
(0096225) git (administrator) 2020-10-24 12:41 |
Branch CR31835 has been deleted by inv. SHA-1: df4f95f132b3192eef399285f93b97b4a2044915 |
![]() |
|||
occt: master 135c2bd9
Timestamp: 2020-10-13 11:31:27 Author: emv Committer: bugmaster [ Details ] [ Diff ] |
0031835: Modeling Algorithms - step by step Boolean common produces bad shape on given three cylinders Fix inconsistency between IntAna_QuadQuadGeo(gp_Cylinder,gp_Cylinder) and AxeOperator::Coplanar(). |
||
mod - src/IntAna/IntAna_QuadQuadGeo.cxx | [ Diff ] [ File ] | ||
add - tests/bugs/modalg_7/bug31835_1 | [ Diff ] [ File ] | ||
add - tests/bugs/modalg_7/bug31835_2 | [ Diff ] [ File ] | ||
![]() |
|||
Date Modified | Username | Field | Change |
2020-10-10 13:09 | Vico Liang | New Issue | |
2020-10-10 13:09 | Vico Liang | Assigned To | => msv |
2020-10-10 13:09 | Vico Liang | File Added: three_cylinder.brep | |
2020-10-10 13:10 | Vico Liang | File Added: three-cylinder.jpg | |
2020-10-10 13:10 | Vico Liang | File Added: two-cylinder-common.jpg | |
2020-10-10 13:10 | Vico Liang | File Added: three-cylinder-common.jpg | |
2020-10-10 13:13 | Vico Liang | Description Updated | View Revisions |
2020-10-12 03:53 | Vico Liang | Severity | minor => block |
2020-10-12 03:53 | Vico Liang | Target Version | 7.6.0* => 7.5.0 |
2020-10-12 03:53 | Vico Liang | Summary | step by step boolean common produces bad shape on given three cylinders => regression issue: step by step boolean common produces bad shape on given three cylinders |
2020-10-12 03:53 | Vico Liang | Description Updated | View Revisions |
2020-10-12 09:30 | msv | Note Added: 0095882 | |
2020-10-12 09:36 | msv | Note Added: 0095883 | |
2020-10-12 09:36 | msv | File Added: Clipboard01.png | |
2020-10-12 15:21 | Vico Liang | Note Added: 0095888 | |
2020-10-12 15:28 | msv | Note Added: 0095889 | |
2020-10-12 15:31 | msv | File Added: Clipboard02.jpg | |
2020-10-12 15:32 | msv | Note Added: 0095890 | |
2020-10-12 17:20 | kgv | Summary | regression issue: step by step boolean common produces bad shape on given three cylinders => [REGRESSION] Modeling Algorithms - step by step Boolean common produces bad shape on given three cylinders |
2020-10-12 17:41 | kgv | Steps to Reproduce Updated | View Revisions |
2020-10-12 17:42 | kgv | Steps to Reproduce Updated | View Revisions |
2020-10-13 09:46 | msv | Note Added: 0095916 | |
2020-10-13 09:47 | msv | Assigned To | msv => emv |
2020-10-13 09:47 | msv | Status | new => assigned |
2020-10-13 11:10 | Vico Liang | Note Added: 0095923 | |
2020-10-13 12:20 | msv | Note Added: 0095926 | |
2020-10-13 14:30 | git | Note Added: 0095930 | |
2020-10-13 14:35 | emv | Note Added: 0095931 | |
2020-10-13 14:35 | emv | Summary | [REGRESSION] Modeling Algorithms - step by step Boolean common produces bad shape on given three cylinders => Modeling Algorithms - step by step Boolean common produces bad shape on given three cylinders |
2020-10-13 14:35 | emv | Steps to Reproduce Updated | View Revisions |
2020-10-13 14:59 | git | Note Added: 0095933 | |
2020-10-13 17:30 | Vico Liang | Note Added: 0095940 | |
2020-10-13 21:35 | git | Note Added: 0095945 | |
2020-10-14 09:29 | emv | Note Added: 0095949 | |
2020-10-14 09:29 | emv | Assigned To | emv => msv |
2020-10-14 09:29 | emv | Status | assigned => resolved |
2020-10-14 10:48 | msv | Note Added: 0095954 | |
2020-10-14 10:48 | msv | Assigned To | msv => bugmaster |
2020-10-14 10:48 | msv | Status | resolved => reviewed |
2020-10-17 13:22 | bugmaster | Note Added: 0096033 | |
2020-10-17 13:22 | bugmaster | Status | reviewed => tested |
2020-10-17 13:29 | bugmaster | Test case number | => bugs/modalg_7/bug31835_1,bug31835_2 |
2020-10-17 13:31 | bugmaster | Changeset attached | => occt master 135c2bd9 |
2020-10-17 13:31 | bugmaster | Status | tested => verified |
2020-10-17 13:31 | bugmaster | Resolution | open => fixed |
2020-10-24 12:41 | git | Note Added: 0096225 | |
2020-12-02 16:22 | emo | Fixed in Version | => 7.5.0 |
2020-12-02 17:13 | emo | Status | verified => closed |
Copyright © 2000 - 2021 MantisBT Team |