OCCT3D
DiscussionsIssue archiveOCCT:Modeling Algorithms

Archived issue #0031836

[REGRESSION] Modeling Algorithms - Boolean cut failed between a Cone and a Torus

CommunityOCCT:Modeling Algorithmsclosed11 public notes

Search issues

Description

The uploaded file Attachment 1 (BREP) have two solid: a cone and a Torus.
It will crash when cut the Torus from the Cone in occt 7.5.0 beta.
It works well in OCCT version 7.4.0.

Steps to reproduce

pload MODELING VISUALIZATION
pcone c 285 100 300
ptorus t 80 20
ttranslate t 0 0 300
bcut r c t
vinit View1
vdisplay -dispMode 1 c t
vaspects t -setColor RED -setTransparency 0.5
vpolygonoffset t 1 2 1
verase c
vdisplay -dispMode 1 r
vaspects r -setColor GREEN
vfit



The code snipped as below:
    BRepAlgoAPI_Cut aCutAlgoAPI;
    aCutAlgoAPI.SetArguments(aArgumentShapes); // Cone
    aCutAlgoAPI.SetTools(aToolShapes);         // Torus
    aCutAlgoAPI.SetFuzzyValue(1e-6);
    aCutAlgoAPI.SetRunParallel(Standard_True);
    aCutAlgoAPI.SetNonDestructive(Standard_True);
    aCutAlgoAPI.SetToFillHistory(Standard_False);
    aCutAlgoAPI.Build();

Public activity

11 archived notes

Participants are labeled by their role within this record.

01Commenter 1
Reverting the following commit:
0024789: BOPAlgo_Builder produces invalid result on planar, conical and spherical surfaces
makes the operation working.
Dear Eugeny, please check what can be done here.
02Commenter 2
Branch [archived branch] has been created by Participant.

[revision removed]


Detailed log of new commits:

Author: emv
Date: Mon Oct 12 15:40:27 2020 +0300

    0031836: regression issue - boolean cut failed between a cone and a Torus.
    
    BOPAlgo_PaveFiller::UpdateExistingPaveBlocks: Add check for closed edge when updating pave block.
03Commenter 3
Draw Harness script reproducing the issue with dynamically constructed torus and cone:

pload MODELING VISUALIZATION
pcone c 285 100 300
ptorus t 80 20
ttranslate t 0 0 300
bcut r c t
vinit View1
vdisplay -dispMode 1 c t
vaspects t -setColor RED -setTransparency 0.5
vpolygonoffset t 1 2 1
verase c
vdisplay -dispMode 1 r
vaspects r -setColor GREEN
vfit


04Commenter 4
Kirill, do you propose to remove the shape from the test base?
05Commenter 3
Eugeny,

> Kirill, do you propose to remove the shape from the test base?
I see no reason adding a file artifact, if problem is reproducible using direct construction - so yes, I propose removing the file.
06Commenter 6
Dear Mikhail, could you please review the git branch CR31836?

Test results- http://jenkins-test-12.nnov.opencascade.com/view/CR31836-master-emv/view/COMPARE/
07Commenter 2
Branch [archived branch] has been updated forcibly by Participant.

[revision removed]
08Commenter 8
Done. File is removed, test case is updated.
09Commenter 9
For integration:
occt - CR31836
products - none
10Commenter 1
Combination -
OCCT branch : [archived branch]
master SHA - [revision removed]
[revision removed]
Products branch : [archived branch] SHA - [revision removed]
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
11Commenter 2
Branch [archived branch] has been deleted by Participant.

[revision removed]

Related records