From: Carsten N. <car...@gm...> - 2011-06-17 19:44:03
|
Hello Gerrit, all, the attached patch changes changedFunctors to receive the change origin as an additional argument - so this may break applications that register changed functors. It also makes sure that during a ChangeList::apply or a RemoteAspect::receiveSync the changed functions of containers are called with ChangedOrigin::Sync instead of the normal ChangedOrigin::Commit. The motivation for this is that when using GPU skinned characters in a cluster where the client renders locally as well, the shader variables already have the correct values set, but GPUSkinningAlgorithm marks them invalid because it receives a changed notification from the Skeleton. Apart from unnecessarily recomputing the values, this produces lots of warnings from ShaderVariableAccess::updateSVariable, because the variable names are not in the map on the remote side (perhaps that's a bug too?). Comments? Cheers, Carsten |