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

(0068757)
git (administrator)
2017-07-26 19:09

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

(0068758)
apv (tester)
2017-07-26 19:10

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

(0078269)
git (administrator)
2018-08-01 16:00

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).

(0078305)
nbv (developer)
2018-08-02 10:15

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.

(0078315)
nbv (developer)
2018-08-02 12:15

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

(0078316)
git (administrator)
2018-08-02 12:19

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:08	nbv	Relationship added	related to 0029972
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

Notes
(0068757) git (administrator) 2017-07-26 19:09	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

(0068758) apv (tester) 2017-07-26 19:10	Problem described in issue is reproduced on current state of OCCT.

(0078269) git (administrator) 2018-08-01 16:00	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).

(0078305) nbv (developer) 2018-08-02 10:15	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.

(0078315) nbv (developer) 2018-08-02 12:15	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

(0078316) git (administrator) 2018-08-02 12:19	Branch CR24429_1 has been deleted by nbv. SHA-1: bd01983fe94ec64210e945edd676ab99777d0cca

Relationships