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

Fpu implementation #115

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

Conversation

Whojo
Copy link

@Whojo Whojo commented Apr 27, 2022

Adds FPU ("First Play Urgency") inspired by Leela-Chess:

  • 2 FPU strategies
    • reduction: subtract the fpu_value to the parent's value
    • absolute: directly replace new nodes value with fpu_value

Note:
FPU is not applied yet at the root node, as in game of src/mcts.jl:explore! for example

Feel free to propose any modifications.

@jonathan-laurent
Copy link
Owner

This looks great. I will merge after we run some experiments on Connect Four. :-)

@oscardssmith
Copy link

based on lc0, relative fpu that is fairly negative is probably a good default.

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.

3 participants