View Issue Details

Related ChangesetsJump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0023849CommunityOCCT:Modeling Algorithmspublic2013-03-22 00:352013-04-29 15:24
ReporterPawel Assigned Toemv 
PriorityimmediateSeverityminor 
Status closedResolutionfixed 
PlatformWindowsOSVC++ 2008 
Product Version6.5.5 
Target Version6.6.0Fixed in Version6.6.0 
Summary0023849: BOP regression (1) - BRepAlgoAPI_Common
DescriptionAlthough 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.
Steps To Reproducepload ALL
restore plane.brep s1
restore segment.brep s2
bop s1 s2
bopcommon result
checkshape result
Additional information
and documentation updates		Set the lower bound for the refined tolerance value in 2d classification algorithm.
TagsNo tags attached.
Test case numberbugs modalg_5(010) bug23849_1, bug23849_2, bug23849_3

Attached Files

  • sample_pair1.zip (420,484 bytes)
  • sample_pair2.zip (414,890 bytes)
  • test_cases.zip (759 bytes)

Relationships

related to 0021762 closedemv Open CASCADE Integration new Boolean Operation algorithm to OCC 

Activities

Pawel

2013-03-22 00:35

developer  

sample_pair1.zip (420,484 bytes)

Pawel

2013-03-22 00:42

developer  

sample_pair2.zip (414,890 bytes)

abv

2013-03-22 07:38

manager   ~0023844

Please fix asap; this is regression caused by one of the very recent integrations (it is Ok on CR23820)

emv

2013-03-22 12:13

developer   ~0023850

This regression caused by the integration of the fix for the issue 23777.

emv

2013-03-22 14:34

developer   ~0023862

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.

pkv

2013-03-22 14:35

developer   ~0023863

No remarks

emv

2013-03-22 14:35

developer  

test_cases.zip (759 bytes)

kgv

2013-03-22 14:40

developer   ~0023864

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()?

emv

2013-03-22 14:56

developer   ~0023865

1.e-7 replaced by Precision::Confusion().

Pawel

2013-03-22 16:59

developer   ~0023870

The patch solves the problem.

mkv

2013-03-22 19:44

tester   ~0023875

Last edited: 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.

Related Changesets

occt: master ce101cac

2013-03-22 10:19:48

emv

Details Diff		 0023849: BOP regression (1) - BRepAlgoAPI_Common.
Fix for the bug.
Set the lower bound for the refined tolerance value in 2d classification algorithm.		 Affected Issues
0023849
mod - src/BRepClass/BRepClass_Intersector.cxx Diff File

Issue History

Date Modified Username Field Change
2013-03-22 00:35 Pawel New Issue
2013-03-22 00:35 Pawel Assigned To => bugmaster
2013-03-22 00:35 Pawel File Added: sample_pair1.zip
2013-03-22 00:37 Pawel Relationship added related to 0021762
2013-03-22 00:37 Pawel Description Updated
2013-03-22 00:40 Pawel Description Updated
2013-03-22 00:42 Pawel File Added: sample_pair2.zip
2013-03-22 00:43 Pawel Summary BOP regression (1) => BOP regression (1) - BRepAlgoAPI_Common
2013-03-22 07:38 abv Note Added: 0023844
2013-03-22 07:38 abv Assigned To bugmaster => emv
2013-03-22 07:38 abv Status new => assigned
2013-03-22 07:38 abv Priority normal => immediate
2013-03-22 07:38 abv Target Version => 6.6.0
2013-03-22 12:13 emv Note Added: 0023850
2013-03-22 14:34 emv Note Added: 0023862
2013-03-22 14:34 emv Assigned To emv => pkv
2013-03-22 14:34 emv Status assigned => resolved
2013-03-22 14:34 emv Additional Information Updated
2013-03-22 14:35 pkv Note Added: 0023863
2013-03-22 14:35 emv File Added: test_cases.zip
2013-03-22 14:35 pkv Assigned To pkv => mkv
2013-03-22 14:35 pkv Status resolved => reviewed
2013-03-22 14:40 kgv Note Added: 0023864
2013-03-22 14:56 emv Note Added: 0023865
2013-03-22 16:59 Pawel Note Added: 0023870
2013-03-22 19:44 mkv Note Added: 0023875
2013-03-22 19:55 mkv Note Edited: 0023875
2013-03-22 19:57 mkv Test case number => bugs modalg_5(010) bug23849_1, bug23849_2, bug23849_3
2013-03-22 19:57 mkv Assigned To mkv => bugmaster
2013-03-22 19:57 mkv Status reviewed => tested
2013-03-25 18:39 emv Changeset attached => occt master ce101cac
2013-03-25 18:39 emv Assigned To bugmaster => emv
2013-03-25 18:39 emv Status tested => verified
2013-03-25 18:39 emv Resolution open => fixed
2013-04-23 13:35 aiv Status verified => closed
2013-04-29 15:24 aiv Fixed in Version => 6.6.0