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

Doesn't work with leela-zero #54

Open
infinity0 opened this issue Jul 13, 2018 · 2 comments
Open

Doesn't work with leela-zero #54

infinity0 opened this issue Jul 13, 2018 · 2 comments

Comments

@infinity0
Copy link

https://github.com/gcp/leela-zero

I see this https://github.com/pzorin/qgo/blob/master/src/gtp/qgtp.cpp#L75

But leela-zero supports both GTP 2 with or without command numbers, and playing by hand works. In fact leela-zero gives back a move but qgo does not update the UI:

QGtp::openGtpSession( "/usr/games/leelaz" , "--gtp -q --weights /home/infinity0/.cache/leela-zero/best-network" )
flush -> protocol_version
** QGtp::slot_readFromStdout():  = 2
** QGtp::waitResponse():  '2'
flush -> boardsize 19
** QGtp::slot_readFromStdout():  = 
** QGtp::waitResponse():  ''
flush -> clear_board
** QGtp::slot_readFromStdout():  = 
** QGtp::waitResponse():  ''
flush -> komi 7.50
** QGtp::slot_readFromStdout():  = 
** QGtp::waitResponse():  ''
flush -> clear_board
** QGtp::slot_readFromStdout():  = 
** QGtp::waitResponse():  ''
flush -> play black D4
** QGtp::slot_readFromStdout():  = 
** QGtp::waitResponse():  ''
flush -> genmove white
** QGtp::slot_readFromStdout():  = Q4
Deleting BoardWindow

I'm also not sure why it says "play black D4" when in the qgo UI it's drawn as D16.

@pzorin
Copy link
Owner

pzorin commented Dec 20, 2018

I've never looked at that part of the code, but it is certainly very wrong given the part of the code that I did look at. I don't have hardware on which leela-zero would run decently, so I won't investigate.

@jackyf
Copy link

jackyf commented Sep 25, 2021

Have had the same issue with fuego (http://fuego.sourceforge.net/). Commenting line 74 in qgtp.cpp indeed solves the issue, just as the issue reporter did anticipate.

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