Archived issue #0030630
Visualization - wrong visualization of shape with mirrored transformation
Description
Current behavior of Std_ShadedShape::fillTriangles() is adopted to visualize shapes with simply applied mirror transformation (mirror invertes solid, so it becames infinity) as a normal shape and, on the contrary, processed mirrored solid with reversed faces as infinity shape.
Steps to reproduce
bugs vis bug30630, bug30630_1, bug30630_2
Public activity
10 archived notes
Participants are labeled by their role within this record.
Branch [archived branch] has been created by Author.
[revision removed]
Detailed log of new commits:
Author: Author
Date: Thu Apr 4 16:07:05 2019 +0300
0030630: Visualization - wrong visualization of shape with mirrored transformation
Solids with mirror location are displayed, according to their normals.
[revision removed]
Detailed log of new commits:
Author: Author
Date: Thu Apr 4 16:07:05 2019 +0300
0030630: Visualization - wrong visualization of shape with mirrored transformation
Solids with mirror location are displayed, according to their normals.
Branch [archived branch] has been updated forcibly by Author.
[revision removed]
[revision removed]
Dear KGV,
could you please review changes in branch CR30630?
It seems, that in old version reversing of triangle nodes sequence by applying transformation with determinant < 0 was not taking in account.
Test base does not affected by this fix:
http://occt-tests/CR30630-master-ika-Products/Debian80-64/diff_summary.html
http://occt-tests/CR30630-master-ika-Products/Windows-64-VC14/diff_summary.html
http://occt-tests/CR30630-master-ika-OCCT/Debian80-64/summary.html
(test bugs fclasses bug7287_6 is not caused by changes of this fix)
http://occt-tests/CR30630-master-ika-OCCT/Windows-64-VC14/diff_summary.html
could you please review changes in branch CR30630?
It seems, that in old version reversing of triangle nodes sequence by applying transformation with determinant < 0 was not taking in account.
Test base does not affected by this fix:
http://occt-tests/CR30630-master-ika-Products/Debian80-64/diff_summary.html
http://occt-tests/CR30630-master-ika-Products/Windows-64-VC14/diff_summary.html
http://occt-tests/CR30630-master-ika-OCCT/Debian80-64/summary.html
(test bugs fclasses bug7287_6 is not caused by changes of this fix)
http://occt-tests/CR30630-master-ika-OCCT/Windows-64-VC14/diff_summary.html
Branch [archived branch] has been updated by Commenter 3.
[revision removed]
Detailed log of new commits:
Author: Commenter 3
Date: Wed Jul 17 19:25:17 2019 +0300
# correct scripts to not use external files
vaspects -setBackFaceColor, added command to set backface color.
[revision removed]
Detailed log of new commits:
Author: Commenter 3
Date: Wed Jul 17 19:25:17 2019 +0300
# correct scripts to not use external files
vaspects -setBackFaceColor, added command to set backface color.
Branch [archived branch] has been updated by Commenter 3.
[revision removed]
Detailed log of new commits:
Author: Commenter 3
Date: Wed Jul 17 20:08:31 2019 +0300
# adjust test cases
[revision removed]
Detailed log of new commits:
Author: Commenter 3
Date: Wed Jul 17 20:08:31 2019 +0300
# adjust test cases
Branch [archived branch] has been created by Commenter 3.
[revision removed]
Detailed log of new commits:
Author: Author
Date: Thu Apr 4 16:07:05 2019 +0300
0030630: Visualization - wrong visualization of shape with mirrored transformation
Solids with mirror location are displayed, according to their normals.
vaspects -setBackFaceColor, added command to set backface color.
[revision removed]
Detailed log of new commits:
Author: Author
Date: Thu Apr 4 16:07:05 2019 +0300
0030630: Visualization - wrong visualization of shape with mirrored transformation
Solids with mirror location are displayed, according to their normals.
vaspects -setBackFaceColor, added command to set backface color.
Reviewed with remarks.
Combination -
OCCT branch : [archived branch]
master SHA - [revision removed]
[revision removed]
Products branch : [archived branch] SHA - [revision removed]
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.
Number of compiler warnings:
No new/fixed warnings
Regressions/Differences/Improvements:
No regressions/differences
CPU differences:
Debian80-64:
OCCT
Total CPU difference: 16229.469999999967 / 16237.640000000089 [-0.05%]
Products
Total CPU difference: 10474.820000000036 / 10517.890000000038 [-0.41%]
Windows-64-VC14:
OCCT
Total CPU difference: 17603.046875 / 17623.078125 [-0.11%]
Products
Total CPU difference: 12080.9375 / 12058.53125 [+0.19%]
Image differences :
No differences that require special attention
Memory differences :
No differences that require special attention
OCCT branch : [archived branch]
master SHA - [revision removed]
[revision removed]
Products branch : [archived branch] SHA - [revision removed]
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.
Number of compiler warnings:
No new/fixed warnings
Regressions/Differences/Improvements:
No regressions/differences
CPU differences:
Debian80-64:
OCCT
Total CPU difference: 16229.469999999967 / 16237.640000000089 [-0.05%]
Products
Total CPU difference: 10474.820000000036 / 10517.890000000038 [-0.41%]
Windows-64-VC14:
OCCT
Total CPU difference: 17603.046875 / 17623.078125 [-0.11%]
Products
Total CPU difference: 12080.9375 / 12058.53125 [+0.19%]
Image differences :
No differences that require special attention
Memory differences :
No differences that require special attention
Branch [archived branch] has been deleted by Participant.
[revision removed]
[revision removed]
Branch [archived branch] has been deleted by Participant.
[revision removed]
[revision removed]