-
Notifications
You must be signed in to change notification settings - Fork 37
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
Add Reset Path Planner UI button #249
Conversation
warnings.warn(f"[{self.name}] Robot has no path planner. Cannot reset.") | ||
return | ||
|
||
if not hasattr(self.path_planner, "reset"): |
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.
@eholum this was inspired by your question on the reset()
method being required. Now it's technically not
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.
Some mighty fine button pushing enabled by this PR.
This PR allows resetting the path planner from the UI.
This means that now if you close a hallway from the UI, you can reset the planner for a robot with PRM/A* and then replan without going right through the door.
Thanks to @muhidabid for setting the foundation with his "Cancel Action" button, I could just copy-paste exactly what he did :)