-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Space-Time AStar #1170
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
Space-Time AStar #1170
Conversation
I didn't find a home for path planning algorithms with dynamic obstacles, let me know if there is somewhere better for this to live :) I used I also didn't look at all of the CI jobs - will come back to address those tomorrow (it's time for sleep where I'm at) |
@SchwartzCode Thank you for great PR!! I loved it.
Creating a new directory under here:
I think it is enough, but please check all CIs are green.
I think all CIs are running, but some are failed, ![]() |
@AtsushiSakai Glad you like the PR! I fixed the CI |
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.
Thank you. @SchwartzCode I have 2 additional requests.
1
Could you please create a PR to add the gifs for this repository?
https://github.com/AtsushiSakai/PythonRoboticsGifs
2
Is it possible to add documentation for this new script based on the manual?
https://atsushisakai.github.io/PythonRobotics/modules/0_getting_started/3_how_to_contribute.html#step-4-write-a-document-about-the-algorithm
PR into the gifs repo: AtsushiSakai/PythonRoboticsGifs#6 |
Added a short doc entry here: 451e545 |
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.
Thank you. I have additional comments and questions. PTAL,
docs/modules/5_path_planning/time_based_grid_search/time_based_grid_search_main.rst
Outdated
Show resolved
Hide resolved
…_grid_search_main.rst
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.
Thank you!!. @SchwartzCode After, all CIs are green, I will merge this.
Reference issue
What does this implement/fix?
This implements SpaceTime A* as outlined in this paper:
https://www.davidsilver.uk/wp-content/uploads/2020/03/coop-path-AIWisdom.pdf
This is a variant of A* that works with dynamic obstacles. I wanted to implement this to serve as a baseline before implementing Safe Interval Path Planning, which is a clever path planning algorithm that greatly reduces the number of node expansions in problems with dynamic obstacles.
Some examples of the SpaceTime A* algorithm running:
Additional information
CheckList