Closed sauk2 closed 3 months ago
Thanks @sauk2 when this gets merged do you want to close https://github.com/gazebosim/gz-sim/pull/2315 ?
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!
🦟 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
Additionally comment out chassis visual for better visibility.
The output after fix looks like this
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.