-
Notifications
You must be signed in to change notification settings - Fork 287
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
Symmetry unittest #1193
Labels
Comments
Thank you for sharing the test code. Let me investigate this. |
I believe PR #1437 fixes this. I've added a regression test based on @JiazhengSun's code. |
azeey
added a commit
to gazebo-forks/dart
that referenced
this issue
Apr 2, 2020
azeey
added a commit
to gazebo-forks/dart
that referenced
this issue
Apr 3, 2020
* Fix issue dartsim#1433 The main fix is in the expression used to integrate velocities. The secondary fix is to update spatial accelerations after integrating velocities and to update both spatial velocities and accelerations after integrating positions. * Add regression test for issue dartsim#1193 * Remove main in test, fix style
Finally resolved by #1437 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This is the unittest I did to check symmetry.
The text was updated successfully, but these errors were encountered: