<div><div dir="auto">Thanks Robert, </div><div dir="auto">   But won’t just simply removing it mean that adding a render leaf will be changing what the current frame is?</div><div dir="auto"> For occlusion nodes I’ve realized that it’s not currently possible to use them within instanced hierarchy without tracking which traversal generated a given query, the cull visitor traversal index (along with other changes) can help disambiguate which draw query it was.</div><div dir="auto">Gedalia</div><div dir="auto"><br></div><div class="gmail_quote"><div>On Sun, May 13, 2018 at 6:55 AM Robert Osfield <<a href="mailto:robert.osfield@gmail.com">robert.osfield@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Gedalia,<br>
<br>
Well spotted, this issue has been in the code for almost two decades<br>
without anyone noticing.  I've removed the duplicate from CullVisitor<br>
and checked this into OSG master and the 3.6 branch.<br>
<br>
Robert.<br>
<br>
On 12 May 2018 at 21:34, Gedalia Pasternak <<a href="mailto:gpasternak@mak.com" target="_blank">gpasternak@mak.com</a>> wrote:<br>
> Both CullVisitor and it's base class NodeVisitor have members named<br>
> _traversalNumber. Yielding duplicate member variables with the same name,<br>
> and inconsistent behavior depending on how you access the class.<br>
> Cull visitor's should be renamed, maybe to _cullTraversalNumber or something<br>
> that wouldn't conflict with the base class. An accessor would be nice as<br>
> well. In the case of NodeVisitor it's what frame it's up to, for cull<br>
> visitor it's what accepted object it's up to (<br>
>    // Otherwise need to create new renderleaf.<br>
>     RenderLeaf* renderleaf = new<br>
> RenderLeaf(drawable,projection,matrix,depth,_traversalNumber++);<br>
> )<br>
><br>
> -Gedalia<br>
><br>
> _______________________________________________<br>
> osg-users mailing list<br>
> <a href="mailto:osg-users@lists.openscenegraph.org" target="_blank">osg-users@lists.openscenegraph.org</a><br>
> <a href="http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org" rel="noreferrer" target="_blank">http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org</a><br>
><br>
_______________________________________________<br>
osg-users mailing list<br>
<a href="mailto:osg-users@lists.openscenegraph.org" target="_blank">osg-users@lists.openscenegraph.org</a><br>
<a href="http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org" rel="noreferrer" target="_blank">http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org</a><br>
</blockquote></div></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">DI-Guy Engineering Lead, <span style="font-family:arial,sans-serif;background-color:rgb(255,255,255)"><span style="font-size:9pt;line-height:13px;color:black">VT MÄK</span></span><br><span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">150 Cambridge Park Drive, </span><span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">3rd Floor, </span><span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">Cambridge, MA 02140</span><span style="font-family:arial,sans-serif;background-color:rgb(255,255,255)"><br></span><br></div></div>