-
Notifications
You must be signed in to change notification settings - Fork 16
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
Pure pursuit ackermann vehicles controller #71
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: ddengster <ed.fan@osrfoundation.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
Signed-off-by: Morgan Quigley <morgan@osrfoundation.org>
Signed-off-by: Morgan Quigley <morgan@osrfoundation.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
Signed-off-by: Morgan Quigley <morgan@osrfoundation.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
Signed-off-by: Morgan Quigley <morgan@osrfoundation.org>
#68) * Slotcar pursues a lookahead point on its path. Add display markers for Ignition. Signed-off-by: Tan Chian Fern <chianfern@openrobotics.org> * Add a short lifetime to waypoint markers Signed-off-by: Tan Chian Fern <chianfern@openrobotics.org> * New velocity compute function for Ackermann and differential drive Signed-off-by: Tan Chian Fern <chianfern@openrobotics.org> * Fix slowdown when turning, draw waypoint markers on path request, and cleanup. Signed-off-by: Tan Chian Fern <chianfern@openrobotics.org> * Handle very small displacements in velocity compute function and clear markers. Signed-off-by: Tan Chian Fern <chianfern@openrobotics.org> * Increase close_enough when holding and rotating towards next waypoint to accommodate slight translation due to rotation. Signed-off-by: Tan Chian Fern <chianfern@openrobotics.org> * Cleanup Signed-off-by: Tan Chian Fern <chianfern@openrobotics.org> * Fix slotcar not moving in gz due to very small velocity changes not taking effect. Signed-off-by: Tan Chian Fern <chianfern@openrobotics.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
Signed-off-by: Luca Della Vedova <luca@openrobotics.org>
aaronchongth
approved these changes
Apr 6, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the delay! The refactor is looks great! I've so far tested on diff drive robots, but I think knowing other projects have been using the ackermann drive, I think we can safely merge this. Thanks!
This was referenced Apr 7, 2022
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New feature implementation
Implemented feature
This PR is a combined work of @codebot and @chianfern and reworks the ackermann controller to use a simpler approach based on pure pursuit.
Since it's a full refactor it will address pre-existing issues, specifically will close #53 and close #55
Implementation description
Together with the pure pursuit logic this PR adds:
<display_markers>
in the slotcar plugin tag.compute_ds
(closes Replace compute_ds with compute_desired_rate_of_change #42 )