Hi Robert,

We've been using the shader composition feature more extensively now, which
works great btw! However, the printing of the entire State object during
every apply of the shader composer is making it nearly impossible to view
any other notifications when lowering the notify level. I can see this
being useful when developing or testing a composer, but I feel having it
always enabled is somewhat obtrusive. Do you have any objections to
removing this or conditionally enabling it using a preprocessor directive?

Cheers,
Farshid


On Tue, May 28, 2013 at 7:36 AM, Robert Osfield <[email protected]>wrote:

> Thanks Farshid, changes look good, now merged and submitted to svn/trunk.
>
> On 9 April 2013 19:12, Farshid Lashkari <[email protected]> wrote:
> > Hi Robert,
> >
> > I just started looking into the shader composition feature. I enabled
> shader
> > composition on the osg::State object and noticed a 6x increase in draw
> time.
> > I figured that the default composer might have performance issues, so I
> > created a custom ShaderComposer that overrides "getOrCreateProgram" and
> does
> > nothing. This still resulted in a performance drop.
> >
> > I looked into the "State::applyShaderComposition" method and noticed
> that it
> > will print debug information to the osg::INFO stream. I commented that
> line
> > out and the performance drop was gone.
> >
> > I'm not sure if the printing was accidentally left in or is meant to be
> > enabled. I've modified the function to only call the print function if
> INFO
> > logging is enabled. I've attached the change.
> >
> > Cheers,
> > Farshid
> >
> > _______________________________________________
> > osg-submissions mailing list
> > [email protected]
> >
> http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
> >
> _______________________________________________
> osg-submissions mailing list
> [email protected]
>
> http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
>
_______________________________________________
osg-submissions mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org

Reply via email to