OCCT3D
DiscussionsIssue archiveOCCT:Modeling Algorithms

Archived issue #0032849

Modeling Algorithms - Intersection algorithm returns incomplete result

Open CASCADEOCCT:Modeling Algorithmsclosed11 public notes

Search issues

Description

For given surfaces intersection algorithm must produce two curves, only one curve is produced.

Steps to reproduce

#Use attached files bug32849_s1, bug32849_s2:

restore bug32849_s1 s1
restore bug32849_s2 s2

intersect ii s1 s2

#result must be ii_1 and ii_2

Public activity

11 archived notes

Participants are labeled by their role within this record.

01Commenter 1
bug32849_s1 (4,338 bytes)   
bug32849_s2 (4,965 bytes)   
02Commenter 1
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Tue Feb 1 12:17:52 2022 +0300

    0032811: Bad result of sweep operation due to Surface Surface Intersect Bug
    
    GeomInt/GeomInt_IntSS.cxx, IntTools/IntTools_FaceFace.cxx - setting deflection 0.01
    for case of two bspline surfaces
    IntPolyh/IntPolyh_Intersection.cxx, IntPolyh/IntPolyh_Intersection.hxx - status IsParallel is added
    
    BndLib/BndLib_Add3dCurve.cxx, IntTools/IntTools_TopolTool.cxx - fix small bugs
    
    Correction of test scripts according to current behavior of algorithms
    
    lowalgos/intss/bug32849 - test case added
03Commenter 3
Branch CR32849 is ready for review.

Issue #0032849 is child of #0032811, which is intended only to solve intersection problem.

Branches for integration:
OCCT - CR32849
Products - not

Test results:
CR32849-master-IFV
04Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
05Commenter 5
Please rename the command OCC24472 into CheckBends, remove the TCL procedure CheckLoops, and change in tests CheckLoops to CheckBends.

Please remove TODO instead of commenting them out.
06Commenter 2
> Please rename the command OCC24472 into CheckBends

Could be `QACheckBends` to keep common prefix for QA commands.
07Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
08Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
09Commenter 9
Branch CR32849 is updated and tested according to remarks

Branches for integration:
OCCT - CR32849
Products - not

Test results:
CR32849-master-IFV
10Commenter 10
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: 18394.330000000256 / 18467.180000000375 [-0.39%]
Products
Total CPU difference: 11708.150000000112 / 11716.630000000077 [-0.07%]
Windows-64-VC14:
OCCT
Total CPU difference: 20348.171875 / 20359.046875 [-0.05%]
Products
Total CPU difference: 13154.5625 / 13140.734375 [+0.11%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
11Commenter 1
Branch [archived branch] has been deleted by Participant.

[revision removed]

Related records