Archived issue #0025430

DRAW Test Harness: dimensions demo fails

Open CASCADEOCCT:DRAWclosed15 public notes

Search issues

Description

Dimensions demo doesn't build some of length and radius dimensions because of inappropriate input geometry.

Public activity

15 archived notes

Participants are labeled by their role within this record.

01Commenter 1
Please consider adding a test case like for other samples, see tests/demo/samples
02Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Wed Oct 29 18:59:24 2014 +0300

    0025430: DRAW Test Harness: dimensions demo fails
    
    - corrected vdimparam help string;
    - added section model file to prevent changing of subshapes order after explode call
03Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
04Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
05Commenter 5
Dear kgv,

could you please review the git branch CR25430?
06Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Thu Oct 30 17:06:58 2014 +0300

    0025430: DRAW Test Harness: dimensions demo fails
    
    - corrected vdimparam help string;
    - added section model file to prevent changing of subshapes order after explode call
07Commenter 7
Dear kgv,

please review rebased branch CR25430_1.
08Commenter 3
Please test the patch.
09Commenter 9
Dear kgv,
could you please rebase CR25430 on current master, there are conflict files.
10Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Thu Oct 30 17:06:58 2014 +0300

    0025430: DRAW Test Harness: dimensions demo fails
    
    - corrected vdimparam help string;
    - added section model file to prevent changing of subshapes order after explode call
11Commenter 11
Dear mkv,
the branch CR25430_1 was rebased on current master and pushed into CR25430_2.
12Commenter 12
Dear Commenter 1,
Branch CR25430_2 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested on Release mode.
[revision removed]

Number of compiler warnings:

occt component :
Linux: 18 (18 on master)
Windows: 0 (0 on master)
MacOS: 199 (199 on master)

products component :
Linux: 11 (11 on master)
Windows: 1 (1 on master)

Regressions/Differences:
No regressions/differences

Testing cases:
http://occt-tests.nnov.opencascade.com/CR25430-2-master-occt/Debian60-64/demo/samples/dimensions.html
http://occt-tests.nnov.opencascade.com/CR25430-2-master-occt/Windows-32-VC10/demo/samples/dimensions.html
demo samples(003) dimensions: OK

Testing on Linux:
occt component :
Total MEMORY difference: 372568824 / 372740520
Total CPU difference: 46668.670000000006 / 48445.730000000076
products component :
Total MEMORY difference: 126055712 / 126153068
Total CPU difference: 16570.099999999962 / 17104.72999999999

Testing on Windows:
occt component :
Total MEMORY difference: 273437848 / 274386716
Total CPU difference: 34221.65625 / 34083.0625
products component :
Total MEMORY difference: 71466340 / 70587484
Total CPU difference: 10382.421875 / 13181.484375

There are following differences in images found by testdiff
http://occt-tests.nnov.opencascade.com/CR25430-2-master-occt/Debian60-64/diff-Debian60-64.html
http://occt-tests.nnov.opencascade.com/CR25430-2-master-occt/Windows-32-VC10/diff-Windows-32-VC10.html
IMAGE demo samples dimensions: dimensions.png differs - new image
13Commenter 2
Branch [archived branch] has been deleted by Participant.

[revision removed]
14Commenter 2
Branch [archived branch] has been deleted by Participant.

[revision removed]
15Commenter 2
Branch [archived branch] has been deleted by Participant.

[revision removed]