Conversation
|
This PR currently depends on the new SwerveDriveOdometry utility proposed in gz-math PR gazebosim/gz-math#718 Should I temporarily disable or stub the odometry part in gz-sim until the math PR is merged to keep CI fully independent? |
|
Thank you for your contribution. It is indeed a useful one. Can we consider targetting main instead of |
|
Great work on implementing the SwerveDrive plugin! This looks like a valuable addition to gz-sim's drive system plugins. A few suggestions:
The implementation follows existing drive system patterns well. Looking forward to seeing this merged! |
Signed-off-by: C88-YQ <1409947012@qq.com>
Signed-off-by: C88-YQ <1409947012@qq.com>
Signed-off-by: C88-YQ <1409947012@qq.com>
Signed-off-by: C88-YQ <1409947012@qq.com>
Signed-off-by: C88-YQ <1409947012@qq.com>
Signed-off-by: C88-YQ <1409947012@qq.com>
Signed-off-by: C88-YQ <1409947012@qq.com>
Signed-off-by: C88-YQ <1409947012@qq.com>
Signed-off-by: C88-YQ <1409947012@qq.com>
1a17a77 to
8685e84
Compare
Signed-off-by: C88-YQ <1409947012@qq.com>
e8f1391 to
a2db4b0
Compare
🎉 New feature
Closes #3364
Related to gazebosim/gz-math#718
Summary
This PR introduces a new SwerveDrive system plugin to gz-sim, providing support for four-wheel independent steering (swerve / 4WISD) kinematics.
The new system:
Kinematic and odometry computation are handled by the
SwerveDriveOdometryutility introduced ingz-math(gazebosim/gz-math#718).This implementation does not modify existing drive systems and is fully backward compatible.
Test it
SwerveDriveOdometry).Checklist
codecheckpassed (See contributing)swerve_drive.mp4
steering_motion.mp4