-
Notifications
You must be signed in to change notification settings - Fork 1
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
Heuristic depth-first search on exploration #25
Comments
@BoolLi what is your status on this? |
@DepthDeluxe Sorry I was busy having interviews this week. I will definitely start working on this over this weekend. I will make sure I have something new by next Monday. |
@DepthDeluxe This line here: What if the robot starts at a position where there is only one way to go? The current logic will have the robot go back, but since it's the first step, there's no history to go back yet. I will try to implement a recursive way to do depth-first search. |
According to the rules, the robot always starts in a spot surrounded by On Wed, Nov 12, 2014 at 8:34 PM, Li Li [email protected] wrote:
|
Then the code won't work if this is the case. In the simulation, the robot always starts at the position where there are at least two ways to go. This is why this bug was not spotted before. I will try to fix it. |
@DepthDeluxe I updated the code and fixed this potential cause of #21, but I haven't thought of an effective way to implement the heuristic yet. I will do it tomorrow. |
@DepthDeluxe Because this fix is not related to the heuristic, I think you can consider merging this branch to master first. |
@BoolLi 👍, send me a PR no that method should only be run once the whole maze is explored. The |
@DepthDeluxe I just meant that in the real competition, are we allowed to scan the whole maze with unlimited amount of time? |
We have 10 minutes to scan and make our best time. This should be enough On Wed, Nov 12, 2014, 10:11 PM Li Li [email protected] wrote:
|
Implement heuristic depth-first search when the robot is exploring. Should be guided to the center but also able to explore an entire maze, regardless of its structure.
The text was updated successfully, but these errors were encountered: