0032301: Visualization, TKOpenGl - depth peeling is broken with OpenGl_Caps::buffersO...

OpenGl_LayerList::renderTransparent() - color mask is now set to write into Alpha channel of depth peeling FBOs.
OpenGl_View::blitBuffers() - avoid using undefined color for glClear(GL_COLOR_BUFFER_BIT) operation.

diff --git a/src/OpenGl/OpenGl_LayerList.cxx b/src/OpenGl/OpenGl_LayerList.cxx
index 563bc6726b40b6eb37465c313f5a42e6f785ee31..53d616fb946033d1a9e68bfb1c0a1f84991cc365 100644 (file)
--- a/src/OpenGl/OpenGl_LayerList.cxx
+++ b/src/OpenGl/OpenGl_LayerList.cxx
@@ -966,6 +966,7 @@ void OpenGl_LayerList::renderTransparent (const Handle(OpenGl_Workspace)&   theW
       // initialize min/max depth buffer
       aGlBlendBackFBO->BindDrawBuffer (aCtx);
       aCtx->SetDrawBuffers (1, THE_DRAW_BUFFERS0);
+      aCtx->SetColorMaskRGBA (NCollection_Vec4<bool> (true)); // force writes into all components, including alpha
       aCtx->core20fwd->glClearColor (0.0f, 0.0f, 0.0f, 0.0f);
       aCtx->core20fwd->glClear (GL_COLOR_BUFFER_BIT);
 
@@ -1122,6 +1123,7 @@ void OpenGl_LayerList::renderTransparent (const Handle(OpenGl_Workspace)&   theW
         theReadDrawFbo->BindBuffer (aCtx);
       }
       aCtx->SetDrawBuffers (1, THE_DRAW_BUFFERS0);
+      aCtx->SetColorMask (true); // update writes into alpha component
       break;
     }
   }

diff --git a/src/OpenGl/OpenGl_View.cxx b/src/OpenGl/OpenGl_View.cxx
index 3f6b63bad8187ce1cb777c7472f4335ea42f1f15..2900289ea5d2ce3b7cf62a603ccf1d55f78a9e56 100644 (file)
--- a/src/OpenGl/OpenGl_View.cxx
+++ b/src/OpenGl/OpenGl_View.cxx
@@ -2558,6 +2558,7 @@ bool OpenGl_View::blitBuffers (OpenGl_FrameBuffer*    theReadFbo,
 
   aCtx->SetColorMaskRGBA (NCollection_Vec4<bool> (true)); // force writes into all components, including alpha
   aCtx->core20fwd->glClearDepth (1.0);
+  aCtx->core20fwd->glClearColor (0.0f, 0.0f, 0.0f, aCtx->caps->buffersOpaqueAlpha ? 1.0f : 0.0f);
   aCtx->core20fwd->glClear (GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT | GL_STENCIL_BUFFER_BIT);
   aCtx->SetColorMask (true); // restore default alpha component write state
 

diff --git a/tests/opengl/data/transparency/oit b/tests/opengl/data/transparency/oit
index 3c4747784584dd48fc2986a31a551af1565eaccd..a6f004731bd293526ce89dc9e56370835adfa2cb 100644 (file)
--- a/tests/opengl/data/transparency/oit
+++ b/tests/opengl/data/transparency/oit
@@ -36,6 +36,11 @@ vdump $imagedir/${casename}_weight.png
 vrenderparams -oit peeling 4
 vdump $imagedir/${casename}_peel.png
 
+# test opaqueAlpha option - result should be the same
+vcaps -opaqueAlpha 1
+vdump $imagedir/${casename}_peel_rgb.png
+vcaps -opaqueAlpha 0
+
 # VTK viewer, just for comparison
 set hasVtk 0
 if { [vdriver -default] == "TKOpenGl" } {