DiscussionsIssue archiveOCCT:Modeling Algorithms

Archived issue #0029501

Modeling Algorithms - Intersection algorithm produces the curve with loop

Open CASCADEOCCT:Modeling Algorithmsassigned12 public notes

Search issues

Description

See Steps To Reproduce and "Attachment 1 (PNG)" attached picture

P.S.

Please pay attention to the fact that this curve is out of surfaces' boundaries

Steps to reproduce

Test bugs modalg_5 bug24472

pload ALL
restore [locate_data_file bug24472_Pipe_1.brep] b1

explode b1 f
copy b1_2 f1
copy b1_3 f2
copy b1_6 f3
mksurface s1 f1
mksurface s2 f2
mksurface s3 f3

intersect i s1 s2

don i_22

Public activity

12 archived notes

Participants are labeled by their role within this record.

01Commenter 1
This problem is reproduced on the current MASTER and has been detected while fixing bug #29494
02Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Fri Oct 26 16:29:48 2018 +0300

    0029501: Intersection algorithm produces the curve with loop
    
    Test case lowalgos/intss/bug24472 has been modified in order to check loops of the resulting intersection curves.
    
    No fix is needed because the issue is not reproduced on MASTER.
03Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
04Commenter 4
Dear Mikhail,

Please review the current state of CR29501 branch.

Test results are here: http://jenkins-test-12.nnov.opencascade.com/view/CR29501-master_NBV/
05Commenter 5
tests/lowalgos/intss/begin
- Why the character "\n" is used instead of direct new line in the comments?
- help checkarea - is this intended here?

Can the test bug28493 be updated to use the new command CheckLoops?
06Commenter 2
Branch [archived branch] has been updated by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Tue Oct 30 09:54:25 2018 +0300

    # Corrections according to remarks

07Commenter 7
I am sorry.

This bug is sstill reproduced on MASTER version. Steps To Reproduce has been updated.

Nevertheless, updated test case should be integrated.
08Commenter 8
Correct the commit message.
09Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
10Commenter 10
Done.
11Commenter 2
Branch [archived branch] has been deleted by Author.

[revision removed]
12Commenter 12
As the bug is not fixed, I leave it open.

Related records