MantisBT - Community
View Issue Details
0023849Community[OCCT] OCCT:Modeling Algorithmspublic2013-03-22 00:352013-04-29 15:24
Pawel 
emv 
immediateminor 
closedfixed 
WindowsVC++ 200832 bit
[OCCT] 6.5.5 
[OCCT] 6.6.0[OCCT] 6.6.0 
bugs modalg_5(010) bug23849_1, bug23849_2, bug23849_3
0023849: BOP regression (1) - BRepAlgoAPI_Common
Although the sample shapes - both are valid - (see attachment) do not overlap a common part is detected by the algorithm.

It seems to me that the distance between both shapes has something to do with the problem because when the shapes are far enough from each other no common part is detected.

The problem exists on the current master (Head: 0022812: Bad performance meshing a face based on extrusion of huge b-spline curve).

I also have a master compiled on 27.02.2013 and the (new) BOP works OK so it seems the problem was introduced after 27.02.
pload ALL
restore plane.brep s1
restore segment.brep s2
bop s1 s2
bopcommon result
checkshape result
Set the lower bound for the refined tolerance value in 2d classification algorithm.
No tags attached.
related to 0021762closed emv Open CASCADE Integration new Boolean Operation algorithm to OCC 
zip sample_pair1.zip (420,484) 2013-03-22 00:35
https://tracker.dev.opencascade.org/
zip sample_pair2.zip (414,890) 2013-03-22 00:42
https://tracker.dev.opencascade.org/
zip test_cases.zip (759) 2013-03-22 14:35
https://tracker.dev.opencascade.org/
Issue History
2013-03-22 00:35PawelNew Issue
2013-03-22 00:35PawelAssigned To => bugmaster
2013-03-22 00:35PawelFile Added: sample_pair1.zip
2013-03-22 00:37PawelRelationship addedrelated to 0021762
2013-03-22 00:37PawelDescription Updatedbug_revision_view_page.php?rev_id=5217#r5217
2013-03-22 00:40PawelDescription Updatedbug_revision_view_page.php?rev_id=5218#r5218
2013-03-22 00:42PawelFile Added: sample_pair2.zip
2013-03-22 00:43PawelSummaryBOP regression (1) => BOP regression (1) - BRepAlgoAPI_Common
2013-03-22 07:38abvNote Added: 0023844
2013-03-22 07:38abvAssigned Tobugmaster => emv
2013-03-22 07:38abvStatusnew => assigned
2013-03-22 07:38abvPrioritynormal => immediate
2013-03-22 07:38abvTarget Version => 6.6.0
2013-03-22 12:08emvRelationship addedchild of 0023777
2013-03-22 12:13emvNote Added: 0023850
2013-03-22 14:34emvNote Added: 0023862
2013-03-22 14:34emvAssigned Toemv => pkv
2013-03-22 14:34emvStatusassigned => resolved
2013-03-22 14:34emvAdditional Information Updatedbug_revision_view_page.php?rev_id=5230#r5230
2013-03-22 14:35pkvNote Added: 0023863
2013-03-22 14:35emvFile Added: test_cases.zip
2013-03-22 14:35pkvAssigned Topkv => mkv
2013-03-22 14:35pkvStatusresolved => reviewed
2013-03-22 14:40kgvNote Added: 0023864
2013-03-22 14:56emvNote Added: 0023865
2013-03-22 16:59PawelNote Added: 0023870
2013-03-22 19:44mkvNote Added: 0023875
2013-03-22 19:55mkvNote Edited: 0023875bug_revision_view_page.php?bugnote_id=23875#r5242
2013-03-22 19:57mkvTest case number => bugs modalg_5(010) bug23849_1, bug23849_2, bug23849_3
2013-03-22 19:57mkvAssigned Tomkv => bugmaster
2013-03-22 19:57mkvStatusreviewed => tested
2013-03-25 18:39emvChangeset attached => occt master ce101cac
2013-03-25 18:39emvAssigned Tobugmaster => emv
2013-03-25 18:39emvStatustested => verified
2013-03-25 18:39emvResolutionopen => fixed
2013-04-23 13:35aivStatusverified => closed
2013-04-29 15:24aivFixed in Version => 6.6.0

Notes
(0023844)
abv   
2013-03-22 07:38   
Please fix asap; this is regression caused by one of the very recent integrations (it is Ok on CR23820)
(0023850)
emv   
2013-03-22 12:13   
This regression caused by the integration of the fix for the issue 23777.
(0023862)
emv   
2013-03-22 14:34   
Git branch CR23849 is ready to be reviewed.
Test cases are attached.
bug23894 - original test case (on BOP algorithm);
bug23849_1 - test case on 2d classifier algorithm.
(0023863)
pkv   
2013-03-22 14:35   
No remarks
(0023864)
kgv   
2013-03-22 14:40   
Dear emv,

could you please comment why magic number 1.e-7 is used within the patch instead of it's named equivalent Precision::Confusion()?
(0023865)
emv   
2013-03-22 14:56   
1.e-7 replaced by Precision::Confusion().
(0023870)
Pawel   
2013-03-22 16:59   
The patch solves the problem.
(0023875)
mkv   
2013-03-22 19:44   
(edited on: 2013-03-22 19:55)
Dear BugMaster,

Branch CR23849 (and products from GIT master) was compiled on Linux and Windows platforms and tested without rebase.
SHA-1: 62830ca87e40e80c8d55d62b9f55c5692aa2b7a6

Number of compiler warnings:

occt component :
Linux: 3 (3 on master)
Windows: 11 (11 on master)

products component :
Linux: 0 (0 on master)
Windows: 50 (50 on master)

Regressions:
No regressions

Improvements:
No improvements

Testing cases:
bugs modalg_5(010) bug23849_1, bug23849_2, bug23849_3 - OK.

Testing on Linux:
Total MEMORY difference: 243633904 / 241816752
Total CPU difference: 16438.40000000011 / 14998.290000000103

Testing on Windows:
Total MEMORY difference: 347734376 / 350384140
Total CPU difference: 20103.640625 / 22701.96875

There are not serious differences in images found by testdiff.