<div dir="ltr">Hi Guy,<br><br><div class="gmail_extra"><div class="gmail_quote">On 20 October 2017 at 15:34, Guy Volckaert <span dir="ltr"><<a href="mailto:guy.volckaert@meggitt.com" target="_blank">guy.volckaert@meggitt.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">My investigation led me to review osg::ComputeBoundsVisitor and I found that it does not consider billboards - i.e. the osg::ComputeBoundsVisitor::<wbr>apply(osg::Billboard& billboard) is missing. In effect, the osg::ComputeBoundsVisitor does not consider the pivot position stored in osg::Billboard::_positionList.<br>
<br>
Is this design intentional? If not, then how would you propose fixing this issue?<br></blockquote><div><br></div><div>This is a design/implementation bug, looking that the implementation it's clearly too simple and misses the special case of handling osg::Billboard.  <br></div><div><br></div><div>To fix the bug a ComputeBoundVisitor::apply(osg::Billboard&) is required, the implementation of this new method would need to iterator through the _positionList and the drawables associated with each position and account for the full rnage of potistions that are possible for each drawables bounding box given the possible rotation and translation required.  The easiest way to do this would be to use the diameter of each drawable child's bounding box for the x,y and z extents.</div><div><br></div><div>I am not in a position to be able to write a fix right away as I have other tasks to complete, but if you can't get to it hopefully next week I'll have a chance to look at it.<br></div><div><br></div><div>Cheers,<br></div><div>Robert.<br></div><div><br></div><div> </div></div></div></div>