DiscussionsIssue archiveOCCT:Modeling Algorithms

Archived issue #0026914

[Regression 7.0alpha] Hang in surface approximation

CommunityOCCT:Modeling Algorithmsclosed13 public notes

Search issues

Description

7.0 preview (master as of Nov 22, 2015) reveals a regression - hang (infinite loop) in surface approximation.
In 6.9.1 approximation takes a second or less in debug mode.

Steps to reproduce

restore a_2773_draw s
approxsurf rs s 5e-5 0 0 15 15 100 0

Public activity

13 archived notes

Participants are labeled by their role within this record.

01Commenter 1
Dear Artem, please check this issue. Try to find what patch could lead to the hanging.
02Commenter 2
Branch [archived branch] has been created by Participant.

[revision removed]


Detailed log of new commits:

Author: azv
Date: Tue Nov 24 14:42:20 2015 +0300

    0026914: [Regression 7.0alpha] Hang in surface approximation
    
    Wrong statement in evaluator of offset surfaces
03Commenter 2
Branch [archived branch] has been updated forcibly by Participant.

[revision removed]
04Commenter 4
Dear Mikhail,

Please, review branch CR26914.
05Commenter 5
Reviewed.
06Commenter 6
Dear Commenter 1,

Branch CR26914 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested.
[revision removed]

Number of compiler warnings:
occt component:
   Linux: 0 (0 on master)
   Windows: 0 (0 on master)
   MasOS: 134 (134 on master)
products component:
   Linux: 39 (39 on master)
   Windows: 0 (0 on master)

Regressions/Differences:
http://occt-tests/CR26914-master-occt-64/Debian70-64/summary.html
http://occt-tests/CR26914-master-occt-64/Windows-64-VC10/summary.html
bugs modalg_4 bug8842_8

Testing on Linux:
Total MEMORY difference: 93051041 / 93521748 [-0.50%]
Total CPU difference: 19768.039999999626 / 19720.20999999994 [+0.24%]

Testing on Windows:
Total MEMORY difference: 58368767 / 58377356 [-0.01%]
Total CPU difference: 18402.252362398973 / 18187.71978719887 [+1.18%]
07Commenter 7
Dear azv,

Branch [archived branch] has been rejected due to:
- regressions/differences/improvements
08Commenter 2
Branch [archived branch] has been updated forcibly by Participant.

[revision removed]
09Commenter 9
Dear Mikhail,

Branch [archived branch] has been updated. Please, review it.
10Commenter 10
Reviewed.
11Commenter 2
Branch [archived branch] has been updated by Participant.

[revision removed]


Detailed log of new commits:

Author: apv
Date: Thu Nov 26 12:34:46 2015 +0300

    Test-case for issue #0026914

12Commenter 12
Dear Commenter 1,

Branch CR26914 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested.
[revision removed]

Number of compiler warnings:
occt component:
   Linux: 0 (0 on master)
   Windows: 0 (0 on master)
   MasOS: 134 (134 on master)
products component:
   Linux: 39 (39 on master)
   Windows: 0 (0 on master)

Regressions/Differences:
Not detected

Testing cases:
bugs modalg_6 bug26914 - OK
http://occt-tests/CR26914-master-occt-64/Debian70-64/bugs/modalg_6/bug26914.html
http://occt-tests/CR26914-master-occt-64/Windows-64-VC10/bugs/modalg_6/bug26914.html

Testing on Linux:
Total MEMORY difference: 93077342 / 93533241 [-0.49%]
Total CPU difference: 19743.599999999497 / 19727.98999999994 [+0.08%]

Testing on Windows:
Total MEMORY difference: 58361757 / 58376918 [-0.03%]
Total CPU difference: 18438.569395198847 / 18194.33422959887 [+1.34%]

13Commenter 2
Branch [archived branch] has been deleted by Participant.

[revision removed]

Related records