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

Make nnue code more idiomatic #3753

Closed

Stefano80 wants to merge 1 commit into official-stockfish:master from Stefano80:refactorNnueMore

Contributor

Stefano80 commented Oct 21, 2021

Make the eval code in the evaluate_nnue.cpp more similar to the rest of the codebase:

remove multiple variable assignment
make if conditions explicit and indent on multiple lines

Tested on STC for non-slowdown

LLR: 2.93 (-2.94,2.94) <-2.50,0.50>
Total: 59032 W: 14834 L: 14751 D: 29447
Ptnml(0-2): 176, 6310, 16459, 6397, 174


          Make code more idiomatic. Bench: 6154836

b62ab0f

vondele added the to be merged label

vondele closed this in

2214fce

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