Skip to content
This repository has been archived by the owner on May 23, 2024. It is now read-only.

Update README.md #194

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

Update README.md #194

wants to merge 1 commit into from

Conversation

kylebakerio
Copy link
Contributor

I think many a newbie has tried to jump into this library and been confused at not understanding how to apply forces. The use of velocity directly is far more predictable than applyImpulse, which makes more sense for programatic use generally. See here for some validation of what I'm talking about, though I think it's also something easy to experience if you try creating a demo following this readme, and may lead to poor initial user experiences with CANNON, as described e.g. here.

Perhaps some glitch demos should be shared here that would be a bit easier to digest for beginners as well?

I think many a newbie has tried to jump into this library and been confused at not understanding how to apply forces. The use of velocity directly is far more predictable than applyImpulse, which makes more sense for programatic use generally. See [here](schteppe/cannon.js#257) for some validation of what I'm talking about, though I think it's also something easy to experience if you try creating a demo following this readme, and may lead to poor initial user experiences with CANNON, as described e.g. [here](https://aframevr.slack.com/archives/C3WGUL4K0/p1617736493000900).

Perhaps some glitch demos should be shared here that would be a bit easier to digest for beginners as well?
@kylebakerio
Copy link
Contributor Author

more confirmation that this approach should be described in the README: #124

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant