Skip to content

Closed kinematic chain#987

Open
yaswanth1701 wants to merge 11 commits into
gazebosim:mainfrom
yaswanth1701:closed-kinematic_chain
Open

Closed kinematic chain#987
yaswanth1701 wants to merge 11 commits into
gazebosim:mainfrom
yaswanth1701:closed-kinematic_chain

Conversation

@yaswanth1701
Copy link
Copy Markdown
Contributor

🎉 New feature

Closes #25

Summary

This feature supports closed kinematic chains for dartsim.

Test it

  • Load any of following models present in gz-physics/test/common_test/worlds/
    • anchored_loop.sdf
    • closed_loop_four_bar_linkage.sdf
    • free_loop.sdf

Video

Closed kinematic loop in dartsim (frequency of oscillation: 0.77) Mimic constrain in bullet-featherstone (frequency of oscillation: 0.83)
Screencast.from.05-22-2026.01.03.48.AM.webm
Screencast.from.05-22-2026.01.03.06.AM.webm

Checklist

  • Signed all commits for DCO
  • Added a screen capture or video to the PR description that demonstrates the feature
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • Updated Bazel files (if adding new files). Created an issue otherwise.
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Comment thread CMakeLists.txt
#============================================================================
# Find gz-cmake
#============================================================================
find_package(gz-cmake REQUIRED)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these look like changes from jetty (gz-physics9). can you rebase the relevant commits onto main?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here is a comparison of an earlier state of your branch against gz-physics9 before it was merged with main:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if you could rebranch from there and then add any missing commits, then you could try squashing it all and rebasing onto main?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the trouble. The issue is resolved now.

Signed-off-by: yaswanth1701 <gonnayaswanth17@gmail.com>
Signed-off-by: yaswanth1701 <gonnayaswanth17@gmail.com>
@yaswanth1701 yaswanth1701 force-pushed the closed-kinematic_chain branch 2 times, most recently from 0827b85 to 4ee9c1d Compare May 21, 2026 22:05
yaswanth1701 and others added 9 commits May 21, 2026 22:11
Signed-off-by: yaswanth1701 <gonnayaswanth17@gmail.com>
Signed-off-by: yaswanth1701 <gonnayaswanth17@gmail.com>
Signed-off-by: yaswanth1701 <gonnayaswanth17@gmail.com>
Signed-off-by: yaswanth1701 <gonnayaswanth17@gmail.com>
Signed-off-by: yaswanth1701 <gonnayaswanth17@gmail.com>
Signed-off-by: yaswanth1701 <gonnayaswanth17@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Inbox

Development

Successfully merging this pull request may close these issues.

Support of closed kinematic chains

3 participants