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.

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

Friction only accounts for dynamic friction #543

Open
Jondolf opened this issue Nov 3, 2024 · 0 comments
Open

Friction only accounts for dynamic friction #543

Jondolf opened this issue Nov 3, 2024 · 0 comments
Labels
A-Dynamics Relates to rigid body dynamics: motion, mass, constraint solving, joints, CCD, and so on C-Bug Something isn't working

Comments

@Jondolf
Copy link
Owner

Jondolf commented Nov 3, 2024

The Friction component stores coefficients of both dynamic and static friction. However, only dynamic (i.e. "sliding") friction is currently accounted for, and the static coefficient does nothing. This is a regression from bevy_xpbd (the predecessor of Avian).

I originally left out static friction because it was causing some serious instability issues, and most other engines like Rapier and Box2D also only consider dynamic friction. However, it could still be nice to have, and it could be that my earlier issues were actually related to #489, which is now fixed.

We should attempt to add back static friction, or if it turns out that it just doesn't work or is otherwise undesirable, we should remove the property from Friction.

@Jondolf Jondolf added C-Bug Something isn't working A-Dynamics Relates to rigid body dynamics: motion, mass, constraint solving, joints, CCD, and so on labels Nov 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Dynamics Relates to rigid body dynamics: motion, mass, constraint solving, joints, CCD, and so on C-Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant