Archived issue #0021494

Intersection between cone and sphere fails

Open CASCADEOCCT:Modeling Algorithmsclosed12 public notes

Description

See Steps To Reproduce.

Steps to reproduce

pcone c 15 0 10
psphere s 10
explode c f
explode s f
mksurface sc c_1
mksurface ss s_1

intersect i sc ss
# there is not an intersection curve, but it must be.

Public activity

12 archived notes

Participants are labeled by their role within this record.

01Commenter 12016-08-02 16:10

Problem is reproduced on current status of master

02Commenter 22016-08-08 15:27

Please process it if time.

03Commenter 32017-10-11 17:27

Problem is reproduced on the current master.

04Commenter 22018-10-29 16:53

Branch [archived branch] has been created by Participant.

[revision removed]

Detailed log of new commits:

Author: nbv
Date: Mon Oct 29 13:34:21 2018 +0300

    0021494: Intersection between cone and sphere fails

    Excess check has been removed.

05Commenter 22018-10-30 13:02

Branch [archived branch] has been updated forcibly by Participant.

[revision removed]

06Commenter 62018-10-30 16:30

Dear Mikhail,

Please review CR21494 branch.

Test results are here: http://jenkins-test-12.nnov.opencascade.com/view/CR21494-master_NBV/

07Commenter 72018-11-02 09:18

Commit message is unclear. Please provide more extended explanation.

The proc CheckIntersectionResult is defined two times in begin and in the test script.

It is better to accept parameter theListOfCurves by value then by name. And upvar will not be needed for it.

08Commenter 22018-11-02 10:02

Branch [archived branch] has been updated forcibly by Participant.

[revision removed]

09Commenter 92018-11-02 10:12

Dear Mikhail,

Please review the current state of the branch CR21494.

Test results are here:

Linux: http://jenkins-test-12.nnov.opencascade.com/view/CR21494-master_NBV/view/TESTING/job/CR21494-master_NBV-OCCT-Debian80-64-opt-test-restart/HTML_20Report/

Windows: http://jenkins-test-12.nnov.opencascade.com/view/CR21494-master_NBV/view/TESTING/job/CR21494-master_NBV-OCCT-Windows-64-VC14-opt-test-restart/HTML_20Report/

10Commenter 102018-11-02 10:20

Reviewed.

11Commenter 12018-11-02 10:39

Combination -
OCCT branch : [archived branch] SHA - [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: 16420.54000000047 / 16441.770000000477 [-0.13%]
Products
Total CPU difference: 7069.500000000021 / 7071.160000000031 [-0.02%]
Windows-64-VC14:
OCCT
Total CPU difference: 17796.625 / 17785.90625 [+0.06%]
Products
Total CPU difference: 8533.640625 / 8535.15625 [-0.02%]

Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention

12Commenter 22018-11-04 14:08

Branch [archived branch] has been deleted by Participant.

[revision removed]