py-stockfish / stockfish

Integrates the Stockfish chess engine with Python (Official fork)
https://py-stockfish.github.io/stockfish/
MIT License
30 stars 9 forks source link

Get rid of engine parameters no longer used #58

Open johndoknjas opened 1 year ago

johndoknjas commented 1 year ago

The self._parameters dict has a few options that don't appear in current Stockfish's options list. Namely:

Continuing to call the stockfish engine with setoption for any of these won't crash the engine. In the terminal it simply says it's not an option, and continues running fine. However, having the user believe these are still options in use of course isn't good. Removing a recent option could be unideal though, if a user is using a version of Stockfish that still had that option.

A middleground would be adjusting self._parameters based off the version of Stockfish the user is using, but maybe this introduces too much complexity into the code.