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

Tong #9

Open
wants to merge 30 commits into
base: gh-pages
Choose a base branch
from
Open

Tong #9

wants to merge 30 commits into from

Conversation

ailyntong
Copy link

@ailyntong ailyntong commented Jul 12, 2016

for mcts maze, I keep getting an error where Expand() where it slowly empties the list of untried moves, but when back at TreePolicy() the node's list of untried moves is not empty. This only happens when removing the last remaining element.

@JoeOsborn
Copy link
Owner

Day 1 looks good! I'll merge this once I've had a chance to look at day 2.

@ailyntong ailyntong changed the title Completed Assignment 1 - explications and hello world Completed Assignment 1 and 2 Jul 13, 2016
@kennyng
Copy link
Collaborator

kennyng commented Jul 14, 2016

For Assignment 2, the functionality seems fine. However, in your check() function, there are some stylistic things such as using "str" as a variable (which is a function in python) or checking if "sm.is_terminal() is False" (which is redundant).

@ailyntong ailyntong changed the title Completed Assignment 1 and 2 Tong Jul 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants