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

black pawns refusing to promote/underpromoting #349

Closed
ppapasaikas opened this issue Apr 18, 2018 · 10 comments
Closed

black pawns refusing to promote/underpromoting #349

ppapasaikas opened this issue Apr 18, 2018 · 10 comments

Comments

@ppapasaikas
Copy link

ppapasaikas commented Apr 18, 2018

There have been multiple reports of a black specific issue where plack pawns do not take a promotion or underpromote sometimes drawing winning positions.
See e.g
http://lczero.org/match_game/98272
http://lczero.org/match_game/99187
https://lichess.org/L9UsfeNB#155

Issue appears both in self-training and actual games.
Seems related to #236, #231

Also possibly (???) related issue: As black not recognizing 3-fold repetition again drawing a winning position.
see e.g:
http://lczero.org/match_game/98422

@smk-kent
Copy link

smk-kent commented Apr 18, 2018

Various match games where this was an issue: 98422, 98277, 98272, 98123, 98496, 98543, 98579, 99187, 99227, 99311, 99441. Notice that the lichess game against IM lovlas with the failed Lucena conversion may also be caused by this, perhaps indirectly. Of those games, 98123 is not about promotion, just the 3-fold issue: black failed to win a QvR endgame, not by the 50moves rule, but by 3-fold. In all those games, black was missing out.

@smk-kent
Copy link

also: http://lczero.org/match_game/98541 the KBNvK endgame, drawn by 3-fold (not 50 moves), again black missing out

@smk-kent
Copy link

in that group there are match games drawn with rook (and more) up against the blank king. There is no incentive in training for quicker wins, so dragging things out appears harmless, and - at least as far as black is concerned - the 3-fold rule is wrongly applied in match games [using 5-fold in training games is madness IMHO]. Somebody counted the plies wrongly, as simple as that. I think this has messed up the NN, and even when fixed will take a while to unlearn.

@ghost
Copy link

ghost commented Apr 18, 2018

@smk-kent Training games use the Stockfish repetition draw code, which uses the 3-fold rule, IIRC. The 5-fold draw rule was -- again IIRC -- only an issue in match games and was since fixed.

Reference: #275

@smk-kent
Copy link

I was told - on the discord chat (by MTGO) - that it was/is used in training games. Whatever it is, it looks as if the counting is not done correctly for black, because I have not found games where white was affected but plenty of games affecting black.

@ghost
Copy link

ghost commented Apr 18, 2018

@smk-kent I just talked to Gary on discord. He's certain that both training and match use 3-fold now (and that training always has).

We discovered that the 50-move rule is being incorrectly implemented as a 25-move draw rule in match games, which caused the behavior in http://lczero.org/match_game/98541. (Note it's not actually a threefold!) This is an issue separate from the promotion, since it stems from the code used to conduct match games. See #350.

@smk-kent
Copy link

that also explains 98123, again 25 moves. At least it does not mess up the net, only distorts match results.

@smk-kent
Copy link

I was told the problem was that black would not see a promotion (other than knight). But in 4 of the games above, 98422, 98543, 98579 and 99187 black could have won anyway, even with knight or no promotions.

@ghost
Copy link

ghost commented Apr 18, 2018

@smk-kent I checked a few of the games you posted (but not all). It seems that Leela was planning on winning those but got stopped by the 25-move bug. The network does not penalize inefficient wins, so it's no surprise it was taking its time.

@killerducky
Copy link
Collaborator

This will be fixed by #351.

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

No branches or pull requests

3 participants