DiscussionsIssue archiveOCCT:Modeling Algorithms

Archived issue #0028690

Section algorithm produces micro edge

Open CASCADEOCCT:Modeling Algorithmsclosed15 public notes

Search issues

Description

This use case is given from the test boolean bsection M8. Two neighboring faces do not share common edge, and thus they are interfering. The section operation produces a micro edge build on one vertex with length 5.62798e-005.

Steps to reproduce

restore Attachment 3 (BREP) a
plane f 20000 0 0 1 0 0
mkface f f
bsection r a f
checknbshapes r -edge 2
checkshape r

Public activity

15 archived notes

Participants are labeled by their role within this record.

01Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Wed Apr 26 11:09:51 2017 +0300

    0028690: Section algorithm produces micro edge
    
    In BOPAlgo_PaveFiller algorithm, prevent creation of vertex-edge interference if this vertex already has interference with one of vertices of this edge. This way we avoid creation of micro edges that are unwanted in the result shape.
02Commenter 2
Please review.
03Commenter 3
Reviewed.
04Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
05Commenter 5
The fix is not ready yet.
06Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
07Commenter 7
I have created the new version of the fix and 3 test cases:
bugs/modalg_6/bug28690,bug28690_1,bug28690_2.
The 2nd test does not work with this fix. It needs further improvement in PerformVE so that to check vertices coincidence after updating their tolerances.

08Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
09Commenter 9
Another case when non-resolved interference between vertices leads to invalid result of Boolean operation:
restore Attachment 1 (BREP) c
explode c
bclearobjects
bcleartools
baddobjects c_1 c_2
bfillds
bbuild result
checkshape result
10Commenter 2
Branch [archived branch] has been updated forcibly by Participant.

[revision removed]
11Commenter 11
The Branch [archived branch] has been rebased on current master. New test case with the Attachment 1 (BREP) input shape has been added.
12Commenter 12
The issue is fixed by the fix for the issue #0028775. The test cases for this issue are added into the patch for #0028775.
13Commenter 13
Dear Commenter 1,
The issue is TESTED in the branch CR28775_2.
14Commenter 1
Modification of code has been integrated with fix for issue CR28775
15Commenter 2
Branch [archived branch] has been deleted by Participant.

[revision removed]

Related records