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
half-move counter seems to be broken. #2
Comments
Oleg Grenrus ***@***.***> writes:
Thanks for this awesome library!
I think I found a small bug in move application, the half move counter doesn't
seem to reset after pawn moves:
*Game.Chess> doPly startpos (move D2 D4)
"rnbqkbnr/pppppppp/8/8/3P4/8/PPP1PPPP/RNBQKBNR b KQkq d3 1 1"
Ahh, good catch. Question is, should it only reset after *pawn* moves,
or after captures as well?
It should be
"rnbqkbnr/pppppppp/8/8/3P4/8/PPP1PPPP/RNBQKBNR b KQkq d3 0 1"
--
CYa,
⡍⠁⠗⠊⠕
|
@mlang The same wikipedia page says
And the page about the Fifty-move rule
|
Thanks for quick fixes! |
Thanks for this awesome library!
I think I found a small bug in move application, the half move counter doesn't seem to reset after pawn moves:
It should be
(FWIW, lichess forget to mark enpassant square, though according to wikipedia
chess.com analysis includes
d3
).EDIT: the counter doesn't seem to reset after captures either.
The text was updated successfully, but these errors were encountered: