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.

Sign up for GitHub

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

Jump to bottom

Dirichlet noise incorrectly added #2

Open
puct9 opened this issue Jan 14, 2021 · 1 comment
Open

Dirichlet noise incorrectly added #2

puct9 opened this issue Jan 14, 2021 · 1 comment

Comments

@puct9
Copy link
Owner

puct9 commented Jan 14, 2021

You're supposed to add the dirichlet noise to the root node upon the first expansion, and not the final probabilities before selecting the move during training.
@puct9
Copy link
Owner Author

puct9 commented Jan 14, 2021

Indeed, that means you shouldn't reuse the tree during training too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@puct9