DiscussionsIssue archiveOCCT:Visualization

Archived issue #0025789

Visualization - get rid of obsolete 2d layers implementation

Open CASCADEOCCT:Visualizationclosed24 public notes

Search issues

Description

As far as 2d layers were implemented through z-layers, it is necessary to remove previous implementation of 2d layers.

Public activity

24 archived notes

Participants are labeled by their role within this record.

01Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Thu Feb 19 13:59:42 2015 +0300

    0025789: Visualization - get rid of obsolete 2d layers implementation
    
    Old implementation of 2d layers through CLayer2d was removed. Draw commands "voverlaytext" and "vlayerline" were updated to work with new implementation of 2d layers.
02Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
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 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
06Commenter 3
Please prepare porting notes within this patch.
07Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Thu Aug 27 14:08:30 2015 +0300

    0025789: Visualization - get rid of obsolete 2d layers implementation
    
    Classes Aspect_Clayer2d, Visual3d_Layer, Visual3d_LayerItem, V3d_LayerMgr, V3d_LayerMgrPointer were deleted.
08Commenter 2
Branch [archived branch] has been updated forcibly by Author.

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

[revision removed]
10Commenter 3
Please test the patch in branch CR25789_2.
11Commenter 2
Branch [archived branch] has been updated forcibly by Participant.

[revision removed]
12Commenter 12
Dear Commenter 1,
Branch CR25789_2 was rebased on branch IR-2015-08-27 of occt git-repository.
[revision removed]
13Commenter 13
Dear Commenter 1,
Branch CR25789_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]

There are following extraction errors:
Linux:
http://jenkins-test-01.nnov.opencascade.com:8080/view/CR25789-2-master/job/CR25789-2-master_prepare_occt_linux/1/parsed_console/
=====> V3d:src

Error : File V3d_LayerMgr.cxx could not be found
Error : Failed during execution
Info : Step src
Error : Step src failed

MacOS:
http://jenkins-test-01.nnov.opencascade.com:8080/view/CR25789-2-master/job/CR25789-2-master_prepare_build_occt_MacOS/1/parsed_console/
=====> V3d:src

Error : File V3d_LayerMgr.cxx could not be found
Error : Failed during execution
Info : Step src
Error : Step src failed
14Commenter 2
Branch [archived branch] has been updated by Commenter 3.

[revision removed]


Detailed log of new commits:

Author: Commenter 3
Date: Fri Aug 28 19:34:00 2015 +0300

    Update V3d/FILES

Author: Commenter 3
Date: Thu Aug 13 18:09:17 2015 +0300

    0026559: Configuration, genproj.tcl - add mandatory C++11 compiler flag to generated Code::Blocks project files

15Commenter 2
Branch [archived branch] has been updated forcibly by Commenter 3.

[revision removed]
16Commenter 2
Branch [archived branch] has been updated forcibly by Commenter 3.

[revision removed]
17Commenter 3
Please test updated patch.
18Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Mon Aug 31 14:43:55 2015 +0300

    Documentation updated.
19Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
20Commenter 20
Dear Commenter 1,
Branch CR25789_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: 15 (15 on master)
Windows: 0 (0 on master)

products component :
Linux: 39 (39 on master)
Windows: 0 (0 on master)

Regressions/Differences/Improvements:
No regressions/differences

Testing cases:
Not needed

Testing on Linux:
occt component :
Total MEMORY difference: 91617093 / 91196283 [+0.46%]
Total CPU difference: 17787.18999999896 / 17162.369999999384 [+3.64%]
products component :
Total MEMORY difference: 25202881 / 25226546 [-0.09%]
Total CPU difference: 7219.8499999999885 / 7039.5099999999975 [+2.56%]

Testing on Windows:
occt component :
Total MEMORY difference: 57038113 / 57044492 [-0.01%]
Total CPU difference: 16036.138395098971 / 16075.981050498936 [-0.25%]
products component :
Total MEMORY difference: 16226544 / 16228570 [-0.01%]
Total CPU difference: 5301.912386399976 / 5334.079792599963 [-0.60%]

There are no differences in images found by testdiff.
21Commenter 21
Dear Commenter 1,
Branch CR25789_3 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: 15 (15 on master)
Windows: 0 (0 on master)

products component :
Linux: 39 (39 on master)
Windows: 0 (0 on master)

Regressions/Differences/Improvements:
No regressions/differences

Testing cases:
Not needed

Testing on Linux:
occt component :
Total MEMORY difference: 91626810 / 91194743 [+0.47%]
Total CPU difference: 17132.689999999286 / 17070.939999999373 [+0.36%]
products component :
Total MEMORY difference: 25227337 / 25216192 [+0.04%]
Total CPU difference: 7136.9199999999855 / 7037.529999999998 [+1.41%]

Testing on Windows:
occt component :
Total MEMORY difference: 57081504 / 57088288 [-0.01%]
Total CPU difference: 16394.753493899083 / 16439.744182298913 [-0.27%]
products component :
Total MEMORY difference: 16220266 / 16222282 [-0.01%]
Total CPU difference: 5336.248206499967 / 5332.831784599964 [+0.06%]

There are no differences in images found by testdiff.
22Commenter 2
Branch [archived branch] has been deleted by Commenter 3.

[revision removed]
23Commenter 2
Branch [archived branch] has been deleted by Commenter 3.

[revision removed]
24Commenter 2
Branch [archived branch] has been deleted by Commenter 3.

[revision removed]

Related records