View Issue Details

IDProjectCategoryView StatusLast Update
0024429CommunityOCCT:Modeling Algorithmspublic2018-08-02 12:19
ReporterRoman Lygin Assigned Tonbv 
PrioritynormalSeverityminor 
Status assignedResolutionopen 
PlatformAOSL 
Product Version6.7.0 
Summary0024429: Cylinder-torus intersection throws an exception and produces no result
DescriptionCylinder-torus intersection throws an exception Standard_OutOfRange (attempt to access an element with index beyond sequence size).
Steps To Reproducerestore s1
restore s2
intersect res s1 s2
TagsNo tags attached.
Test case numberbugs modalg_7 bug24429

Attached Files

  • surfaces.zip (351 bytes)
  • 0001-0024429-Cylinder-torus-intersection-throws-an-except.patch (17,881 bytes)

Activities

Roman Lygin

2013-12-02 22:48

developer  

surfaces.zip (351 bytes)

git

2017-07-26 19:09

administrator   ~0068757

Branch CR24429 has been created by apv.

SHA-1: 33faae3b3bed01cf48f5e02fd6026535ece0db93


Detailed log of new commits:

Author: apv
Date: Wed Jul 26 19:09:13 2017 +0300

    Test for 0024429: Cylinder-torus intersection throws an exception and produces no result

apv

2017-07-26 19:10

tester   ~0068758

Problem described in issue is reproduced on current state of OCCT.

git

2018-08-01 16:00

administrator   ~0078269

Branch CR24429_1 has been created by nbv.

SHA-1: bd01983fe94ec64210e945edd676ab99777d0cca


Detailed log of new commits:

Author: nbv
Date: Wed Aug 1 15:49:36 2018 +0300

    0024429: Cylinder-torus intersection throws an exception and produces no result
    
    1. The reason of exception has been eliminated.
    
    2. Interfaces of the following methods have been changed in the classes Contap_TheIWalking and IntPatch_TheIWalking: Perform(...), TestArretPassage(...) and ComputeCloseLine(...).
    
    <break!>
    
    Main changes are:
    
    1. IntWalk_IWalking::TestArretPassage(...).
      a) Anisotropic normalization of 2D-coordinates has been eliminated.
      b) Algorithm of check of validity of WLine has been improved.
    
    2. IntWalk_IWalking::ComputeCloseLine(...). Check of indices of the sequence of points has been added (to avoid exceptions).

nbv

2018-08-02 10:15

developer   ~0078305

Dear All,

The exception in the intersection algorithm will be fixed in the branch CR29972*. However, the intersection result will be wrong (not closed), see the updated test case in the branch CR24429_1.

The branch CR24429_1 fixes the problem in intersection algorithm completely but leads to many regressions (including overload by CPU-limit). So, it is not ready to be integrated.

nbv

2018-08-02 12:14

developer  

0001-0024429-Cylinder-torus-intersection-throws-an-except.patch (17,881 bytes)

nbv

2018-08-02 12:15

developer   ~0078315

The patch of the fix CR24429_1 has been uploaded in the file "0001-0024429-Cylinder-torus-intersection-throws-an-except.patch". It is based on remotes/origin/IR-2018-07-27

git

2018-08-02 12:19

administrator   ~0078316

Branch CR24429_1 has been deleted by nbv.

SHA-1: bd01983fe94ec64210e945edd676ab99777d0cca

Issue History

Date Modified Username Field Change
2013-12-02 22:48 Roman Lygin New Issue
2013-12-02 22:48 Roman Lygin Assigned To => ifv
2013-12-02 22:48 Roman Lygin File Added: surfaces.zip
2017-07-26 19:09 git Note Added: 0068757
2017-07-26 19:10 apv Test case number => bugs modalg_7 bug24429
2017-07-26 19:10 apv Note Added: 0068758
2018-08-01 13:09 nbv Assigned To ifv => nbv
2018-08-01 13:09 nbv Status new => feedback
2018-08-01 16:00 git Note Added: 0078269
2018-08-02 10:08 nbv Status feedback => assigned
2018-08-02 10:15 nbv Note Added: 0078305
2018-08-02 12:14 nbv File Added: 0001-0024429-Cylinder-torus-intersection-throws-an-except.patch
2018-08-02 12:15 nbv Note Added: 0078315
2018-08-02 12:19 git Note Added: 0078316