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

Don't use NNUE in variants #147

Closed
benwh1 opened this issue Dec 22, 2020 · 3 comments
Closed

Don't use NNUE in variants #147

benwh1 opened this issue Dec 22, 2020 · 3 comments
Labels

Comments

@benwh1
Copy link

benwh1 commented Dec 22, 2020

Using a network trained on classical chess to analyse variant games doesn't make any sense. In particular, the atomic analysis on lichess is much worse than it was before the update (I haven't tested the other variants). For example see this position. The analysis says the position is slightly better for black, but it's actually completely winning for white. Running the latest stockfish on my computer, it immediately says +3 for white with NNUE disabled, but takes 10 seconds to go above 0 with NNUE enabled.

@bharrisau
Copy link
Contributor

Variants are handled by a Stockfish branch AFAIK. ddugovic/Stockfish#158 is looking for good benchmarking examples to make sure there aren't any performance regressions - this is probably a good example.

@niklasf niklasf added the bug label Dec 22, 2020
@niklasf
Copy link
Member

niklasf commented Dec 22, 2020

Oh, wow, I was (apparently incorrectly) assuming multi-variant Stockfish would not use NNUE for variants.

@niklasf
Copy link
Member

niklasf commented Dec 22, 2020

Released in v2.1.3.

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

No branches or pull requests

3 participants