DiscussionsIssue archiveOCCT:Visualization

Archived issue #0029602

Visualization, TKOpenGl - Size Culling is not properly handled within Perspective projection

Open CASCADEOCCT:Visualizationclosed6 public notes

Search issues

Description

Moving camera Eye+Center within Perspective projection does not lead to expected results of Size Culling - only Scale has effect.

Steps to reproduce

test v3d glsl cullsize 1
vviewparams -eye 16.8333 -9.08333 10.0833 -at 7.5 0.25 0.75
if { [vreadpixel 125 170 rgb name] != "BLACK" } { puts "Error!" }

Public activity

6 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 Mar 22 13:05:06 2018 +0300

    0029602: Visualization, TKOpenGl - Size Culling is not properly handled within Perspective projection
    
    OpenGl_BVHTreeSelector::isTooSmall() now takes into account distance
    the distance between camera Eye and object Center
    within perspective projection.
02Author
Patch is ready for review.
03Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
04Commenter 4
Branch CR29602 reviewed without remarks, ready for testing.
05Commenter 1
Combination -
OCCT branch : [archived branch] SHA - [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:
Debian70-64:
OCCT
Total CPU difference: 18133.56999999989 / 18107.97999999989 [+0.14%]
Products
Total CPU difference: 7432.340000000017 / 7427.109999999995 [+0.07%]
Windows-64-VC10:
OCCT
Total CPU difference: 17999.052177798683 / 17842.61437499857 [+0.88%]
Products
Total CPU difference: 8116.576028999973 / 8148.135031299978 [-0.39%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
06Commenter 2
Branch [archived branch] has been deleted by Author.

[revision removed]