The options appear to have totally changed

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

The options appear to have totally changed #3

Closed llimllib closed 9 years ago

llimllib commented 9 years ago

pystockfish fails right now with ValueError: No such option: Mobility (Middle Game), and indeed it appears that the available options have completely changed. Would you accept it if I made a PR to change the option names?

iamjarret commented 9 years ago

It's frustrating that the new version isn't backward compatible. It'll take me a week to review the changes, but your fix would be greatly appreciated.

llimllib commented 9 years ago

What's cool too is that they are randomly renaming options, so for example the version on my system uses "Contempt Factor" but github HEAD uses "Contempt".

I think we ought to change it so that an invalid option doesn't throw an exception, as they seem to be pretty in flux and you can still run stockfish if the option you tried to set doesn't exist. Does that seem reasonable to you?