-
Notifications
You must be signed in to change notification settings - Fork 125
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.
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
Troika Physics #15
base: main
Are you sure you want to change the base?
Troika Physics #15
Conversation
…ow cube map textures to be set
…eJS geometry and constructor parameters
…nges to `physics` properties when physics are running. Split out physics examples.
} | ||
} | ||
|
||
// FIXME should this attempt to force the parent world into continuousRender mode? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think if you add a this.notifyWorld('needsRender')
call in the worker response handler, that should give a similar result as continousRender.
Alternatively, you could consider having the physics manager set up its own timing loop rather than hooking in to onBeforeRender.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the suggestions, that definitely sounds cleaner. I'll investigate those options next.
… code portability and remove unused components.
…plify loading and bundling
…rent physics engines. Default package `main` files to ammo.js implementation. Update all worker code to use WorkerModules
…tring prefix to ammo worker IDs.
…ule-ified Ammo.js output file. Add TODO for finishing ASM.js fallback
…, refactor worker messages to use transferable, start new MouseInteractionExample, integrate adapted AmmoDebugDrawer
…ges, debug drawer
New
troika-physics
module, powered by ammo.js.Example Usage