Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve output verbosity and format, and tune Skill Level as an example #1

Merged
merged 3 commits into from
Dec 4, 2016

Conversation

ddugovic
Copy link

@ddugovic ddugovic commented Dec 4, 2016

During initialization, print engine output including warnings/errors which may occur (such as "No such option: Aggressiveness")

Output shows interval using "x ± step" for readability

Since Stockfish no longer has an Aggressiveness parameter, change example to tune Skill Level

@ianfab
Copy link
Owner

ianfab commented Dec 4, 2016

At this point I should probably say that I did not know much more of Perl than its name when I forked the SPSA tuner and it has not changed much since then. I just added code using syntax I concluded from the context.

Anyway, the changes look fine to me as far as I can judge. Just a very minor point: I would prefer not to change the path of the executable in the example as this usually is the file name after having compiled Stockfish.

@ddugovic
Copy link
Author

ddugovic commented Dec 4, 2016

Thanks for pointing that out. I have changed back the engine name... oh, and the engine path which I had forgotten I changed.

@ianfab ianfab merged commit 8a39598 into ianfab:master Dec 4, 2016
@ddugovic ddugovic deleted the ddugovic branch December 4, 2016 16:23
@ddugovic
Copy link
Author

ddugovic commented Dec 4, 2016

I admit that over a decade ago I edited one or two Perl files and I too am learning Perl. I ran SPSA many times until I got a basic familiarity with it (and the "simulate" mode was very helpful in that discovery process).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants