View Issue Details

IDProjectCategoryView StatusLast Update
0023794Open CASCADEOCCT:Data Exchangepublic2014-02-07 14:06
Reporterika Assigned Toika  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
PlatformAOSL 
Product Version6.5.4 
Target Version6.6.0Fixed in Version6.6.0 
Summary0023794: Too small precision leads to instability
DescriptionToo small precision tol2(10^-36) in line
if( devia>tol2 || (dist2-dist)>dPreci || ddist>1.e-3*step ) in file ShapeConstruct_ProjectCurveOnSurface.cxx in function ApproxPCurve leads to different results on Windows and Mandriva2010
TagsNo tags attached.
Test case numberNot needed

Relationships

parent of 0023439 closedika Integration of test grid "xigs" into the new testing system 

Activities

ika

2013-02-25 12:42

manager   ~0023471

tol2 (10^-36) was changed to dPreci (10^-18)

Branch CR23794 is ready to be reviewed.

Dear GKA,
Please review.

gka

2013-02-25 12:44

developer   ~0023472

Branch CR23794 is ready to test

mkv

2013-02-28 12:27

tester   ~0023520

Dear BugMaster,

Branch CR23794 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
SHA-1: fdf64b9d9d77972d141931a283df241db8cc32ea

Number of compiler warnings:

occt component :
Linux: 3 (3 on master)
Windows: 39 (39 on master)

products component :
Linux: 9 (9 on master)
Windows: 50 (50 on master)

Regressions:
http://occt-tests/CR23794-master-occt/Windows-32-VC9/summary.html
bugs modalg_2 (007) bug453_2 - New value of area (only on Windows)
bugs heal(017) bug210 - checkshape

Improvements:
No improvements

Testing cases:
Not needed

Testing on Linux:
Total MEMORY difference: 239545808 / 239512028
Total CPU difference: 19242.310000000776 / 16118.16999999981

Testing on Windows:
Total MEMORY difference: 343571716 / 343334684
Total CPU difference: 15184.828125 / 16592.875

There are not serious differences in images found by testdiff.

ika

2013-03-01 15:21

manager   ~0023551

Last edited: 2013-03-01 15:42

bugs modalg_2 bug453_2: it fails not only on branch with this fix, but on master, without one too(please check it).

bugs heal bug210: incident, which leads to checkshape error in this test has not been detected before fix, although it existed, so test case was changed by adding TODO(now this test should be BAD), and pushed into Branch CR23794.

mkv

2013-03-01 17:00

tester   ~0023552

Dear BugMaster,

This fix is TESTED.

Related Changesets

occt: master 9b308cd3

2013-03-07 09:21:34

ika

Details Diff
0023794: Too small precision leads to instability
tol2 (10^-36) was changed to dPreci (10^-18)
test case was changed by adding TODO
Affected Issues
0023794
mod - src/ShapeConstruct/ShapeConstruct_ProjectCurveOnSurface.cxx Diff File
mod - tests/bugs/heal/bug210 Diff File

Issue History

Date Modified Username Field Change
2013-02-25 11:36 ika New Issue
2013-02-25 11:36 ika Assigned To => ika
2013-02-25 12:42 ika Note Added: 0023471
2013-02-25 12:42 ika Assigned To ika => gka
2013-02-25 12:42 ika Status new => resolved
2013-02-25 12:43 gka Status resolved => reviewed
2013-02-25 12:44 gka Note Added: 0023472
2013-02-26 17:56 ika Relationship added parent of 0023439
2013-02-27 19:53 mkv Assigned To gka => mkv
2013-02-28 12:27 mkv Note Added: 0023520
2013-02-28 12:28 mkv Test case number => Not needed
2013-02-28 12:28 mkv Assigned To mkv => ika
2013-02-28 12:28 mkv Status reviewed => assigned
2013-03-01 15:21 ika Note Added: 0023551
2013-03-01 15:21 ika Assigned To ika => mkv
2013-03-01 15:21 ika Status assigned => feedback
2013-03-01 15:42 ika Note Edited: 0023551
2013-03-01 17:00 mkv Note Added: 0023552
2013-03-01 17:00 mkv Assigned To mkv => bugmaster
2013-03-01 17:00 mkv Status feedback => tested
2013-03-11 17:38 ika Changeset attached => occt master 9b308cd3
2013-03-11 17:38 ika Assigned To bugmaster => ika
2013-03-11 17:38 ika Status tested => verified
2013-03-11 17:38 ika Resolution open => fixed
2013-04-23 13:35 aiv Status verified => closed
2013-04-29 15:23 aiv Fixed in Version => 6.6.0