Indeed there is an error with the 2D filter actuator: it stays enabled and runs on each frame, which results in the shader program to be recompiled each time... big performance waste.

Hopefully the fix is extremely simple.
Replace the "return true" at line 77 of SCA_2DFilterActuator.cpp by:

// once the filter is in place, no need to update it
return false;

That's it.
Can you test the fix will all the shader stuff that you have?

Benoit so far I tested and this fix helps.
I want to run some more tests though.

And I asked Mike (Pan) to test with some of his shaders as well. As soon as we get a position I let you know.

Thanks a lot !

I did not receive more feedback so I assume the fix is working.

Fix commited in trunk and ticket closed

Sorry about the lack of feedback. I was trying to talk with zaghaghi about that, but I couldnt find him online since then.

Your fix solves the problem of the 2dfilter been called every logic tic.
We are still re-creating the textures every frame though, instead of only copying. But this is something that was introduced in the last zaghaghi commit (17469) - http://lists.blender.org/pipermail/bf-blender-cvs/2008-November/016678.html

It was introduced as a bug fix, so the better would be to talk with zaghaghi directly.

Thanks Benoit

The commit that you refer to is not on the same file: I patched SCA_2DFilterActuator.cpp while the commit is about RAS_2DFilterManager.cpp.

I am pretty sure of my fix but I will pass the ticker to Zaghaghi to let him comment on that.

hi benoit,

thank you for fixing the bad bug ;), i should take care of these mistakes.

This task was automatically closed as archived as part of migration, because the project or tracker this task belonged to is no longer active.