Skip to content

[bugfix] check bestmove only for multipv 1 lines#999

Merged
Disservin merged 1 commit intoDisservin:masterfrom
robertnurnberg:bestmove-bugfix
Apr 2, 2026
Merged

[bugfix] check bestmove only for multipv 1 lines#999
Disservin merged 1 commit intoDisservin:masterfrom
robertnurnberg:bestmove-bugfix

Conversation

@robertnurnberg
Copy link
Copy Markdown
Contributor

Fixes #998.

Master:

> ./fastchess -engine name=sf-dev1 cmd=./stockfish option.'MultiPV'=2 -engine name=sf-dev2 cmd=./stockfish option.'Skill Level'=10 -each tc=0.2+0.01 -rounds 5 -concurrency -1 -openings file=UHO_Lichess_4852_v1.epd format=epd > fc_master.txt
> grep Warning fc_master.txt | grep -c sf-dev1 
573
> grep Warning fc_master.txt | grep -c sf-dev2 
215

Patch:

> ./fastchess -engine name=sf-dev1 cmd=./stockfish option.'MultiPV'=2 -engine name=sf-dev2 cmd=./stockfish option.'Skill Level'=10 -each tc=0.2+0.01 -rounds 5 -concurrency -1 -openings file=UHO_Lichess_4852_v1.epd format=epd > fc_patch.txt
> grep Warning fc_patch.txt | grep -c sf-dev1
0
> grep Warning fc_patch.txt | grep -c sf-dev2
259

So the output shows that both correctly report bestmove mismatch for sf-dev with skill levels, while only master reports wrong bestmove for sf-dev in multiPV mode.

@Disservin Disservin merged commit 541aef8 into Disservin:master Apr 2, 2026
18 checks passed
@robertnurnberg robertnurnberg deleted the bestmove-bugfix branch April 2, 2026 12:26
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.

Fastchess appears to use a multipv > 1 to warn against PV not matching bestmove

2 participants