Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Performance improvement for static or sleeping bodies. #903

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Performance improvement for static or sleeping bodies. #903

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 23 additions & 10 deletions src/core/Engine.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -132,8 +132,10 @@ var Body = require('../body/Body');

// get lists of all bodies and constraints, no matter what composites they are in
var allBodies = Composite.allBodies(world),
nonStaticBodies = [],
nonSleepingBodies = [],
allConstraints = Composite.allConstraints(world);

// @if DEBUG
// reset metrics logging
Metrics.reset(engine.metrics);
Expand All @@ -142,19 +144,29 @@ var Body = require('../body/Body');
// if sleeping enabled, call the sleeping controller
if (engine.enableSleeping)
Sleeping.update(allBodies, timing.timeScale);


// Fill lists with static and sleeping bodies
for (var i = 0; i < allBodies.length; i++) {
var body = allBodies[i];
if(!body.isStatic) {
nonStaticBodies.push(body);
if(!body.isSleeping)
nonSleepingBodies.push(body);
}
}

// applies gravity to all bodies
Engine._bodiesApplyGravity(allBodies, world.gravity);
Engine._bodiesApplyGravity(nonSleepingBodies, world.gravity);

// update all body position and rotation by integration
Engine._bodiesUpdate(allBodies, delta, timing.timeScale, correction, world.bounds);
Engine._bodiesUpdate(nonSleepingBodies, delta, timing.timeScale, correction, world.bounds);

// update all constraints (first pass)
Constraint.preSolveAll(allBodies);
Constraint.preSolveAll(nonStaticBodies);
for (i = 0; i < engine.constraintIterations; i++) {
Constraint.solveAll(allConstraints, timing.timeScale);
}
Constraint.postSolveAll(allBodies);
Constraint.postSolveAll(nonStaticBodies);

// broadphase pass: find potential collision pairs
if (broadphase.controller) {
Expand All @@ -163,7 +175,8 @@ var Body = require('../body/Body');
broadphase.controller.clear(broadphase);

// update the grid buckets based on current bodies
broadphase.controller.update(broadphase, allBodies, engine, world.isModified);
broadphase.controller.update(broadphase,
world.isModified ? allBodies : nonSleepingBodies, engine, world.isModified);
broadphasePairs = broadphase.pairsList;
} else {
// if no broadphase set, we just pass all bodies
Expand Down Expand Up @@ -200,11 +213,11 @@ var Body = require('../body/Body');
Resolver.postSolvePosition(allBodies);

// update all constraints (second pass)
Constraint.preSolveAll(allBodies);
Constraint.preSolveAll(nonStaticBodies);
for (i = 0; i < engine.constraintIterations; i++) {
Constraint.solveAll(allConstraints, timing.timeScale);
}
Constraint.postSolveAll(allBodies);
Constraint.postSolveAll(nonStaticBodies);

// iteratively resolve velocity between collisions
Resolver.preSolveVelocity(pairs.list);
Expand All @@ -225,7 +238,7 @@ var Body = require('../body/Body');
// @endif

// clear force buffers
Engine._bodiesClearForces(allBodies);
Engine._bodiesClearForces(nonSleepingBodies);

Events.trigger(engine, 'afterUpdate', event);

Expand Down