Skip to content
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

Dodat uslov za IDSF. #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

amilivojevic
Copy link

Postavljeno rešenje za IDFS ne radi u slučaju kada robot ne može da se kreće po dijagonalama.
Desi se da je lista čvorova states_list neprazna i svi čvorovi u listi imaju dubinu veću od max_depth, zatim poziv funkcije self.select_state(states_list) ne vrati ništa i u narednoj liniji se od tog ništa pokušava naći unique_hash.

@jocicmarko
Copy link
Contributor

Dobro zapažanje. Problem je zapravo to što IDFS u svakom produbljivanju/iteraciji obilazi čvorove koje je već obišao u prethodnoj iteraciji.
Što se tiče ovog PR, neću ga merge-ovati jer ovde imaš i urađene zadatke na originalnim fajlovima, ali ću ostaviti PR otvoren da ostali mogu da vide.

Pozdrav i hvala,
Marko

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants