MantisBT - Community
View Issue Details
0030395Community[OCCT] OCCT:Modeling Algorithmspublic2018-12-05 12:162018-12-15 20:28
Matthias 
apn 
normalminor 
verifiedfixed 
WindowsVC++ 201564 bit
[OCCT] 7.3.0 
 
bugs modalg_7 bug30395
0030395: DRAW bop: Some sub-shapes of some of the argument become connected through other shapes and the argument became self-interfered.
In this specific example the boolean operation returns an empty compound.

With "bdrawwarnshapes 1" the critical shape "ws_1_1" is returned after "bop". It is a compound containing two edges (red in the given picture).

It is unclear why there is a critical shape in this example and why bopcommon returns no result.
DRAW script:

bdrawwarnshapes 1
restore s1.brep s1
restore fds.brep fds
bop s1 fds
bopcommon r
No tags attached.
png pic.png (144,261) 2018-12-05 12:16
https://tracker.dev.opencascade.org/
? s1.brep (52,166) 2018-12-05 12:17
https://tracker.dev.opencascade.org/
? fds.brep (237,692) 2018-12-05 12:17
https://tracker.dev.opencascade.org/
Issue History
2018-12-05 12:16MatthiasNew Issue
2018-12-05 12:16MatthiasAssigned To => msv
2018-12-05 12:16MatthiasFile Added: pic.png
2018-12-05 12:17MatthiasFile Added: s1.brep
2018-12-05 12:17MatthiasFile Added: fds.brep
2018-12-05 12:39emvNote Added: 0081387
2018-12-05 12:41emvAssigned Tomsv => Matthias
2018-12-05 12:41emvStatusnew => feedback
2018-12-11 15:00MatthiasNote Added: 0081465
2018-12-11 15:00MatthiasAssigned ToMatthias => emv
2018-12-11 15:28emvAssigned Toemv => bugmaster
2018-12-11 15:28emvNote Added: 0081467
2018-12-11 16:20msvNote Added: 0081469
2018-12-11 18:24gitNote Added: 0081471
2018-12-11 18:24apnTest case number => bugs modalg_7 bug30395
2018-12-11 18:24apnNote Added: 0081472
2018-12-11 18:24apnAssigned Tobugmaster => emv
2018-12-12 07:47emvNote Added: 0081474
2018-12-12 07:47emvAssigned Toemv => apn
2018-12-12 07:47emvStatusfeedback => assigned
2018-12-12 08:10gitNote Added: 0081475
2018-12-12 08:12emvNote Added: 0081476
2018-12-12 08:12emvStatusassigned => resolved
2018-12-12 11:53apnAssigned Toapn => bugmaster
2018-12-12 11:53apnStatusresolved => reviewed
2018-12-12 11:53apnStatusreviewed => tested
2018-12-15 20:28apnChangeset attached => occt master d4db985b
2018-12-15 20:28apnAssigned Tobugmaster => apn
2018-12-15 20:28apnStatustested => verified
2018-12-15 20:28apnResolutionopen => fixed

Notes
(0081387)
emv   
2018-12-05 12:39   
Dear Matthias,

Could you please check if the problem is reproduced in the current master? I am unable to reproduce it there. But I confirm that the problem is reproduced in OCCT 7.3.0.
(0081465)
Matthias   
2018-12-11 15:00   
I'm unable to reproduce it in the current master, too. It seems to be okay there.
(0081467)
emv   
2018-12-11 15:28   
Dear bugmaster, could you please add the test case and close the issue?
(0081469)
msv   
2018-12-11 16:20   
After test preparation, please pass to EMV for review.
(0081471)
git   
2018-12-11 18:24   
Branch CR30395 has been created by apn.

SHA-1: b00586e9a65e1f09d11f4043e4c7f6e2990d6cb6


Detailed log of new commits:

Author: apn
Date: Tue Dec 11 18:21:20 2018 +0300

    0030395: DRAW bop: Some sub-shapes of some of the argument become connected through other shapes and the argument became self-interfered.
    
    Test case bugs/modalg_7/bug30395 was added
(0081472)
apn   
2018-12-11 18:24   
Dear emv, could you please review test case in CR30395.
(0081474)
emv   
2018-12-12 07:47   
- Please add check for the number of shapes in the result. Don't need to check for the number of vertices and edges, as their number may change from fix to fix, but check for the number of wires, faces, shells and solids.
- Please add also the check for the volume property of the result.
- It will also be good to perform all other boolean operations (fuse and two cuts) directly in this case, to avoid missing regressions in other operations on these shapes. The same checks will be required for the results of these operations. But displaying only the result of Common is enough here, no need to display other results.
(0081475)
git   
2018-12-12 08:10   
Branch CR30395 has been updated forcibly by emv.

SHA-1: 15fb5ed9f3a1f9927d80644ff190c7fbe56c9555
(0081476)
emv   
2018-12-12 08:12   
I've edited the test case myself, so please check and integrate.