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

🐊tinybrain: katago deser & illegal move error #456

Open
Terkwood opened this issue Nov 20, 2020 · 2 comments
Open

🐊tinybrain: katago deser & illegal move error #456

Terkwood opened this issue Nov 20, 2020 · 2 comments
Labels
bug Something isn't working

Comments

@Terkwood
Copy link
Owner

Terkwood commented Nov 20, 2020

KataGo deser fails

Found these errors logged in tinybrain:

  [ redact ERROR tinybrain::katago] Deser error in katago response: Error("missing field `moveInfos`", line: 1, column: 100)
     raw: {"error":"Illegal move 10: G5","field":"moves","id":"aaaa-redact_12_WHITE"}
 
  [ redact ERROR tinybrain::katago] Deser error in katago response: Error("missing field `moveInfos`", line: 1, column: 103)
     raw: {"error":"Illegal move 147: O15","field":"moves","id":"bbbb-redact_149_BLACK"}

To reproduce, check journalctl after some time passes

why is it missing that field?

We should improve the logging of these failed requests

what is the illegal move error?

does this cause system downtime?

It seems so. But the downtime can also occur without the illegal move error. See #510

@Terkwood Terkwood added the bug Something isn't working label Nov 20, 2020
@Terkwood Terkwood pinned this issue Nov 21, 2020
@Terkwood Terkwood unpinned this issue Dec 3, 2020
@Terkwood
Copy link
Owner Author

Terkwood commented Mar 25, 2021

More of these around Mar 17.

I'm not sure if this is the cause, but, the service stopped functioning and had to be restarted ⚰️ We also changed local DNS config at the hosting site, around the same time -- but that seems unlikely to impact an existing WebSocket connection

I have some private logs dated Mar 25 which can be examined if necessary

@Terkwood Terkwood pinned this issue Mar 25, 2021
@Terkwood
Copy link
Owner Author

KataGo unresponsive again, this time after doing another DNS switch at its hosting site!

@Terkwood Terkwood changed the title tinybrain: katago throws illegal move error 🐊tinybrain: katago throws illegal move error Apr 4, 2021
@Terkwood Terkwood changed the title 🐊tinybrain: katago throws illegal move error 🐊tinybrain: katago deser can cause system downtime Jun 12, 2021
@Terkwood Terkwood changed the title 🐊tinybrain: katago deser can cause system downtime 🐊tinybrain: katago deser & illegal move error Jun 16, 2021
@Terkwood Terkwood unpinned this issue Jul 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant