iamjarret / pystockfish

Implements an easy-to-use Engine class to integrates the Stockfish chess engine with Python
GNU General Public License v3.0
30 stars 13 forks source link

Match.run() fix, add tests, parses bestmove's info #13

Closed pafnucy closed 7 years ago

pafnucy commented 7 years ago

Fixes https://github.com/iamjarret/pystockfish/issues/12

Didn't experiment with ponder, but I think it should be fine in proposed setup. I changed order of code, so break in the move loop occurs when there is mate, no matter what is value of ponder.

If you have any ideas for further testing, please share.

pafnucy commented 7 years ago

Thanx for feedback. This pull request also fixes https://github.com/iamjarret/pystockfish/issues/8, although I didn't split "pv" field, so it is a string instead of list. Didn't notice https://github.com/iamjarret/pystockfish/pull/9 earlier, so I guess I somewhat duplicated work.

By the way, I think both https://github.com/iamjarret/pystockfish/issues/5 and https://github.com/iamjarret/pystockfish/issues/10 can be closed.

Could you drop me an email?