Skip to content
This repository has been archived by the owner on Apr 23, 2024. It is now read-only.

server is double-sending moves in some cases #15

Open
nelhage opened this issue May 4, 2016 · 3 comments
Open

server is double-sending moves in some cases #15

nelhage opened this issue May 4, 2016 · 3 comments

Comments

@nelhage
Copy link
Contributor

nelhage commented May 4, 2016

I have a bot that's logging all games on the server for analysis. In some cases where White plays a move and then the game ends (I can't tell from logs if this is due to time or resignation or what), it double-sends the opening move.

2016/05/04 10:46:26 < GameList Add Game#309 TakticianBot vs onewhohides, 5x5, 1200, 0 half-moves played, TakticianBot to move
2016/05/04 10:46:26 > Observe 309
2016/05/04 10:46:26 < Observe Game#309 TakticianBot vs onewhohides, 5x5, 1200, 1 half-moves played, onewhohides to move
2016/05/04 10:46:26 < Game#309 P A1
2016/05/04 10:46:26 < Game#309 Time 1200 -2216321
2016/05/04 10:46:26 < Game#309 P A1
2016/05/04 10:46:26 < Game#309 Time 1200 -2216321
2016/05/04 10:46:26 < GameList Remove Game#309 TakticianBot vs onewhohides, 5x5, 1200, 1 half-moves played, onewhohides to move
2016/05/04 10:46:26 < Game#309 Over 1-0
2016/05/04 10:46:26 < GameList Remove Game#309 TakticianBot vs onewhohides, 5x5, 1200, 1 half-moves played, onewhohides to move

There's also some weirdness with the time, which is being reported as -2216321 here…

@chaitu236
Copy link
Owner

This looks like a synchronization issue. Currently, when I add a new spectator, I send the move list, but if the one of the players moves at the same time, there is a possibility of sending the move twice.
Thanks for the bug report, I'd not noticed it before.

@chaitu236
Copy link
Owner

Oh and recording games at your end might be redundant as I plan to release all the games.

@nelhage
Copy link
Contributor Author

nelhage commented May 5, 2016

Yeah, I figured you'd get around to it, but it was easy and I wanted them
for TakticianBot analysis. Thanks for the response!

On Wed, May 4, 2016, 17:03 Chaitanya Vadrevu [email protected]
wrote:

Oh and recording games at your end might be redundant as I plan to release
all the games.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#15 (comment)

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

No branches or pull requests

2 participants