DiscussionsIssue archiveOCCT:Data Exchange

Archived issue #0024206

Exception is raised in the STEP loopback tests.

Open CASCADEOCCT:Data Exchangeclosed15 public notes

Search issues

Description

Exception is raised in the STEP loopback tests due to not initialized variables in ShapeFix_EdgeProjAux and ShapeAnalysis_Curve classes.

Steps to reproduce

Following tests are failed only in release mode:
de step_2 P4
de step_3 A4 B8 C8
de step_4 I2

Public activity

15 archived notes

Participants are labeled by their role within this record.

01Commenter 1
Branch CR24206 is ready to be reviewed.
02Commenter 2
I observe these fails on OCCT built with vc10 (except is that in group step_3 I have B9 failed, not B8)
03Commenter 3
Reviewed, please test (branch CR24206_1, rebased on current master and squashed to one commit)
04Commenter 4
Dear Commenter 1,

Branch CR24206_1 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
[revision removed]

Number of compiler warnings:

occt component :
Linux: 428 (424 on master) - New additional warnings!
Windows: 10 (9 on master) - New additional warning!

products component :
Linux: 189 (189 on master)
Windows: 287 (287 on master)

Regressions/Differences:
http://occt-tests/CR24206-1-master-occt/Debian60-64/bugs/xde/bug22805.html
bugs xde(005) bug22805: FAILED
05Commenter 5
Please provide list of new warnings
06Commenter 6
07Commenter 7
Linux:

ShapeAnalysis_Curve.cxx:182, GNU Compiler 4 (gcc), Priority: Normal
unused variable ‘distmin’
08Commenter 8
Branch CR24206_1 is ready to be reviewed.
09Commenter 9
No remarks, please test
10Commenter 10
Dear Commenter 1,

Branch CR24206_1 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
[revision removed]

Number of compiler warnings:

occt component :
Linux: 370 (370 on master)

Windows: 7 (6 on master)
http://jenkins-test-02.nnov.opencascade.com:8080/user/mnt/my-views/view/A_mnt_warnings/job/mnt-CR24206_1-master_build_occt_windows/1/warnings27Result/package.-1346939154/
New additional warning:
ShapeFix_EdgeProjAux.cxx:392, MSBuild, Priority: Normal
'isClosed' : local variable is initialized but not referenced


products component :
Linux: 189 (191 on master)
Windows: 287 (287 on master)

Regressions/Differences:
http://jenkins-test-02.nnov.opencascade.com:8080/user/mnt/my-views/view/CR24206_1/job/mnt-CR24206_1-master_products_tests_linux_start/label=Debian60-64,tests_group=sat,tests_subgroup=002/2/HTML_Report/?
http://jenkins-test-02.nnov.opencascade.com:8080/user/mnt/my-views/view/CR24206_1/job/mnt-CR24206_1-master_products_tests_windows_start/label=windows_test,tests_group=sat,tests_subgroup=002/2/HTML_Report/?
sat doc_2(002) A4, A5, A9, F7, H1

Testing cases:
http://occt-tests/CR24206-1-master-occt/Debian60-64/de/
http://occt-tests/CR24206-1-master-occt/Windows-32-VC9/de/
de step_2(005) P4; de step_3(006) A4,B8,C8; de step_4(007) I2 - OK.

Testing on Linux:
Total MEMORY difference: 353213064 / 353452256
Total CPU difference: 40340.010000000126 / 43493.42000000097

Testing on Windows:
Total MEMORY difference: 407446124 / 411317668
Total CPU difference: 31965.453125 / 37715.5625

There are not differences in images found by testdiff.
11Commenter 11
Warning "ShapeFix_EdgeProjAux.cxx:392, MSBuild, Priority: Normal
'isClosed' : local variable is initialized but not referenced" was corrected.

Result of test cases sat doc_2(002) A4, A5, A9, F7, H1 are the same as in current master of Products.
12Commenter 12
Dear gka,
could you please check branch CR24206_1.
Last commit was write on 14/10/2013, but last testing was on 16/10/2013.

13Commenter 13
Modification is put in branch CR24206_1
14Commenter 14
Dear Commenter 1,

Branch CR24206_1 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
[revision removed]

Number of compiler warnings:

occt component :
Linux: 368 (368 on master)
Windows: 6 (6 on master)

products component :
Linux: 190 (190 on master)
Windows: 287 (287 on master)

Regressions/Differences:
sat doc_2(002) A4, A5, A9, F7, H1 - these test cases are corrected

Testing cases:
http://occt-tests/CR24206-1-master-occt/Debian60-64/de/
http://occt-tests/CR24206-1-master-occt/Windows-32-VC9/de/
de step_2(005) P4; de step_3(006) A4,B8,C8; de step_4(007) I2 - OK.

Testing on Linux:
Total MEMORY difference: 353476912 / 353823372
Total CPU difference: 42954.93000000104 / 42954.25000000077

Testing on Windows:
Total MEMORY difference: 407566572 / 407073520
Total CPU difference: 33366.921875 / 37415.03125

There are not differences in images found by testdiff.

15Commenter 15
Dear Commenter 1,

Test cases
sat doc_2(002) A4, A5, A9, F7, H1
are corrected and pushed in new git branch CR24206_Products.