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

LockedAxes break ExternalTorque #492

Closed
unpairedbracket opened this issue Aug 18, 2024 · 0 comments · Fixed by #486
Closed

LockedAxes break ExternalTorque #492

unpairedbracket opened this issue Aug 18, 2024 · 0 comments · Fixed by #486
Labels
A-Dynamics Relates to rigid body dynamics: motion, mass, constraint solving, joints, CCD, and so on C-Bug Something isn't working

Comments

@unpairedbracket
Copy link
Contributor

If an entity has a LockedAxes component and has any of the rotational axes locked, applying an ExternalTorque to that entity has no effect (even if the axis of the torque applied is not one of the locked axes).

I've adapted the reproduction from issue #474 to demonstrate. I had thought this problem was introduced in the fix to that issue, but in reproducing I discovered that versions of it exist in 0.1.0, 0.1.1 and 0.1.2. The Cargo.toml includes an option to use my proposed fix branch, which corresponds to PR #486. The reproduction can be found here: https://github.com/unpairedbracket/av-repro

I've kind of done the issue reporting/reproduction/PR fix backwards here - sorry it's taken so long to put a reproduction together!

@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 Aug 19, 2024
@Jondolf Jondolf linked a pull request Aug 19, 2024 that will close this issue
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

Successfully merging a pull request may close this issue.

2 participants