Skip to content
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

Fixed turning error in ackermann steering #2342

Merged
merged 1 commit into from
Mar 27, 2024

Conversation

sauk2
Copy link
Contributor

@sauk2 sauk2 commented Mar 23, 2024

🦟 Bug fix

Fixes #2314

Summary

This is a fix to the error seen in Ackermann Steering's <steering_only> mode. The steps to reproduce this error are described in issue #2314.

The solution to this was changing signs in these equations.

Steps to check the fix

  1. Add the following tag in the example SDF file of Ackermann Steering plugin.
  <steering_only>True</steering_only>
  1. Build and source the workspace
  2. Launch using the following command
  gz sim ackermann_steering.sdf
  1. Publish the following
  gz topic -t "/model/vehicle_blue/steer_angle" -m gz.msgs.Double -p "data: 0.4"

Additionally comment out chassis visual for better visibility.

The output after fix looks like this

image

Checklist

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.

@bperseghetti
Copy link
Member

Thanks @sauk2 when this gets merged do you want to close #2315 ?

@sauk2
Copy link
Contributor Author

sauk2 commented Mar 27, 2024

Thanks @sauk2 when this gets merged do you want to close #2315 ?

@bperseghetti Yes, I will go ahead and close #2315 once this gets merged. Thank you for your input and feedback on these PRs!

@bperseghetti bperseghetti merged commit 1d5114b into gazebosim:gz-sim8 Mar 27, 2024
6 of 9 checks passed
@sauk2 sauk2 deleted the fix/ackermann-turning branch March 27, 2024 17:32
GauravKumar9920 pushed a commit to GauravKumar9920/gz-sim that referenced this pull request Mar 30, 2024
This is a fix to the error seen in Ackermann Steering's <steering_only> mode. The steps to reproduce this error are described in issue gazebosim#2314.

Signed-off-by: Saurabh Kamat <[email protected]>
Signed-off-by: Gaurav Kumar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎵 harmonic Gazebo Harmonic
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Ackermann Steering plugin's steering only mode shows opposite behaviour than expected
3 participants