-
Notifications
You must be signed in to change notification settings - Fork 41
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
bullet-featherstone: Add Kinematic feature #618
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Ian Chen <ichen@openrobotics.org>
Signed-off-by: Ian Chen <ichen@openrobotics.org>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #618 +/- ##
==========================================
+ Coverage 78.32% 78.36% +0.04%
==========================================
Files 140 141 +1
Lines 8069 8136 +67
==========================================
+ Hits 6320 6376 +56
- Misses 1749 1760 +11 ☔ View full report in Codecov by Sentry. |
I think we need more documentation on what it means for a I'm currently looking at the documentation for ODE and bullet for inspiration: |
also for dart: https://github.com/dartsim/dart/blob/main/dart/dynamics/Joint.hpp#L178-L187 |
Signed-off-by: Ian Chen <ichen@openrobotics.org>
I expanded the SDF description in gazebosim/sdformat#1462 and updated the Link |
🎉 New feature
Summary
Note: bullet version >=3.07 is need for this feature. On Jammy, the version is 3.06 so you'll need to build bullet from source. Noble should come with 3.24
Initial support for kinematic mode in bullet-featherstone implementation.
<kinematic>
SDF value sets a link to kinematic iftrue
.Not supported yet - setting velocity to a kinematic link or to a joint connecting kinematic links.
Test it
Added tests to make sure we can make base and non-base links kinematic.
You can also test with gz-sim.
Example: the
base
andupper_link
links are set to kinematic in the pendulum_links.sdf.patch
Launch the world with bullet-featherstone plugin:
Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining
Signed-off-by
messages.