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

move silently fails to send #417

Open
raylu opened this issue Apr 2, 2024 · 0 comments
Open

move silently fails to send #417

raylu opened this issue Apr 2, 2024 · 0 comments

Comments

@raylu
Copy link
Contributor

raylu commented Apr 2, 2024

my bot crashed during this game https://online-go.com/game/62948925. here are the last bit of logs

Mar 30 08:13:35   Game.ts:581          [game 62948925] Playing h2 { x: 7, y: 7, text: 'h2', resign: false, pass: false }
Mar 30 08:13:38   Game.ts:249          [game 62948925] Opponent played j3
Mar 30 08:13:38   Game.ts:446          [game 62948925] Acquiring main bot instance
Mar 30 08:13:38   Game.ts:456          [bot  2377:3481078] [game 62948925] Acquired bot instance
Mar 30 08:13:38   Game.ts:511          [bot  2377:3481078] Generating move for game 62948925
Mar 30 08:13:38   Game.ts:512          [game 62948925] genmove white
Mar 30 08:13:39   Game.ts:369          [game 62948925] Releasing bot(s)
Mar 30 08:13:39   Game.ts:625          [game 62948925] Min move time was  1500 ms and we only took  510 ms. Waiting  990 ms before sending move
Mar 30 08:13:40   Game.ts:581          [game 62948925] Playing j2 { x: 8, y: 7, text: 'j2', resign: false, pass: false }
Mar 30 08:13:40   Game.ts:249          [game 62948925] Opponent played h4
Mar 30 08:13:40   Game.ts:446          [game 62948925] Acquiring main bot instance
Mar 30 08:13:40   Game.ts:456          [bot  2377:3481078] [game 62948925] Acquired bot instance
Mar 30 08:13:40   Game.ts:511          [bot  2377:3481078] Generating move for game 62948925
Mar 30 08:13:40   Game.ts:512          [game 62948925] genmove white
Mar 30 08:13:41   Game.ts:369          [game 62948925] Releasing bot(s)
Mar 30 08:13:41   Game.ts:625          [game 62948925] Min move time was  1500 ms and we only took  568 ms. Waiting  932 ms before sending move
Mar 30 08:13:42   Game.ts:581          [game 62948925] Playing h8 { x: 7, y: 1, text: 'h8', resign: false, pass: false }
Mar 30 08:14:12   Game.ts:136          [game 62948925] gamedata     W   𝖌𝖔 𝖕𝖗𝖔66  [9x9]
Mar 30 08:14:12   Game.ts:446          [game 62948925] Acquiring main bot instance
Mar 30 08:14:12   Game.ts:456          [bot  2377:3481078] [game 62948925] Acquired bot instance
Mar 30 08:14:12   Game.ts:511          [bot  2377:3481078] Generating move for game 62948925
Mar 30 08:14:12   Game.ts:512          [game 62948925] genmove white

so you can see we recognized the Opponent played h4 and then we logged Playing h8. but the last move OGS has is h4

the next line in my logfile is my bot crashing because it asserts that it's black's turn

my diagnosis is that we call

socket.send("game/move", {

which just calls https://github.com/online-go/goban/blob/311bdb5b4e1b62524d26e945c00602effa8dc9bb/src/GobanSocket.ts#L352 with no cb
we should instead send a callback with an error handler, possibly via GobanSocket.sendPromise

I suspect this might be the same cause as #409

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

1 participant