glinscott / leela-chess

**MOVED TO https://github.com/LeelaChessZero/leela-chess ** A chess adaption of GCP's Leela Zero
http://lczero.org
GNU General Public License v3.0
760 stars 299 forks source link

Change isready to call UpdateNetwork so backends initialize #686

Closed killerducky closed 6 years ago

killerducky commented 6 years ago

I think UCI expects isready to block and do all long operations, so for us that should include backend initialization. From the examples I think all setoptions should be done before this so we will know the network the user wants by now.

I made engine::UpdateNetwork public instead of private to do this, let me know if you want to do it differently. I also called EnsureOptionsSent, not sure if we need that or other things.

killerducky commented 6 years ago

Move to https://github.com/LeelaChessZero/lc0/pull/23