You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently the benchmark function returns the "Nodes/second" output line after running the bench command. However, generally the "Nodes searched" line is of interest as well. So maybe add a parameter to benchmark allowing the user to specify what piece of info they want, and consider just returning the int value (rather than a string of the whole line).
On a separate note, consider making params a default Optional param equaling None. If it is None, just send the bench command without anything else to Stockfish.
The text was updated successfully, but these errors were encountered:
Currently the
benchmark
function returns the "Nodes/second" output line after running thebench
command. However, generally the "Nodes searched" line is of interest as well. So maybe add a parameter tobenchmark
allowing the user to specify what piece of info they want, and consider just returning the int value (rather than a string of the whole line).On a separate note, consider making
params
a default Optional param equalingNone
. If it isNone
, just send thebench
command without anything else to Stockfish.The text was updated successfully, but these errors were encountered: