Skip to content

Conversation

@Mugen87
Copy link
Collaborator

@Mugen87 Mugen87 commented Dec 29, 2018

Simplified the example by just using a single mixer variable instead of an array. Notice how the following previous pattern is problematic:

for ( var i = 0; i < mixers.length; i ++ ) {

    mixers[ i ].update( clock.getDelta() );

}

If you actually have multiple mixers in your scene, clock.getDelta() will produce an invalid time delta value for the second and all subsequent mixers. It would be better to call clock.getDelta() once and reuse the same time delta value for all mixers.

@Mugen87
Copy link
Collaborator Author

Mugen87 commented Dec 29, 2018

@mrdoob mrdoob added this to the r100 milestone Dec 31, 2018
@mrdoob mrdoob merged commit 7e77d64 into mrdoob:dev Dec 31, 2018
@mrdoob
Copy link
Owner

mrdoob commented Dec 31, 2018

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants