DiscussionsIssue archiveOCCT:Modeling Algorithms

Archived issue #0027553

Modeling Algorithms - BRepFeat_SplitShape returns self-intersecting result since OCC 7.0.0

CommunityOCCT:Modeling Algorithmsassigned10 public notes

Search issues

Description

For the appended case BRepFeat_SplitShape returns an invalid result for one subshape since OCCT 7.0.0. In OCCT 6.9.1 there was no problem.
The problem cannot be fixed by fixshape command.

The problem occurs for the third face of the initial shell.
In the reproducing script the splitshapes commands differ for OCC 7.0.0 and 6.9.1 in the last argument because the order of shapes in the result of bopsection changed.

Steps to reproduce

restore Attachment 1 (BREP) s

whatis s

explode s

whatis s_1

whatis s_2

tolerance s_1

tolerance s_2

checkshape s_1

checkshape s_2

explode s_2

brunparallel 1

bop s_1 s_2

bopsection r

explode r

explode s_2


# OCC 7.0.0
splitshape r1 s_2_1 s_2_1 r_4

splitshape r2 s_2_2 s_2_2 r_2

splitshape r3 s_2_3 s_2_3 r_1

splitshape r4 s_2_4 s_2_4 r_3


checkshape r1

checkshape r2

checkshape r3
#=> invalid !!!
checkshape r4



# OCC 6.9.1
splitshape r1 s_2_1 s_2_1 r_3

splitshape r2 s_2_2 s_2_2 r_4

splitshape r3 s_2_3 s_2_3 r_2

splitshape r4 s_2_4 s_2_4 r_1


checkshape r1

checkshape r2

checkshape r3

checkshape r4

Public activity

10 archived notes

Participants are labeled by their role within this record.

01Commenter 1
The problem is not in SplitShape algorithm. The section produces the curve with self-intersection. See the bug #0027648.
02Commenter 2
Dear Commenter 1, please test if it is reproduced.
03Commenter 3
Branch [archived branch] has been created by Participant.

[revision removed]


Detailed log of new commits:

Author: mkv
Date: Fri Jul 21 18:55:57 2017 +0300

    Test for 0027553: BRepFeat_SplitShape returns self-intersecting result since OCC 7.0.0
04Commenter 4
Dear msv,
problem described in issue is reproduced on current state of OCCT.
05Commenter 5
Probably fixed by #29511.
06Commenter 6
Please create a test case.
07Commenter 7
Sorry, test is present already.
08Commenter 8
Just need to review and integrate.
09Commenter 9
Please review the branch CR27553.
10Commenter 10
1. Rebase on current master.
2. Remove from the script unneeded commands that check input shapes. Leave only necessary for test commands.
3. Add commands to check sprops of the result.
4. Remove TODO if the bug is really fixed.
5. Create job on jenkins to check the command on both platforms.

Related records