google / google/brax

Add gyroscopic term to physics integrator

Open
#48 5 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Jupyter Notebook
Stars
3.2k
Forks
349
PR merge metrics
No merged PRs in 30d

Description

Looking at the code in `integrator.py`, I did not find the right equation of evolution of the angular velocity (`ang` in the code)

According to Featherstone (and a derivation of mine and my university's textbook), the correct equations of motions should be :

With tau being the external torques, omega the angular velocity of the body, and I the inertia matrix in the local frame.

I guess is does not make much of a difference when most degrees of freedom are constrained, but clean math should be better.

Any plans to include this version of the equations in Brax ? And if no, can you give me some pointers as to how one might go and implement that ?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.