qgo icon indicating copy to clipboard operation
qgo copied to clipboard

Doesn't work with leela-zero

Open infinity0 opened this issue 6 years ago • 2 comments

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.

infinity0 avatar Jul 13 '18 04:07 infinity0

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.

pzorin avatar Dec 20 '18 20:12 pzorin

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.

jackyf avatar Sep 25 '21 15:09 jackyf