-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Small todos from the next dev cycle #5567
Comments
Also check if this CI step is really required, https://github.com/official-stockfish/Stockfish/blob/master/.github/workflows/upload_binaries.yml#L40. I have a feeling that the runner already has git and zip, and make shouldnt even be needed. |
Rename the binary |
though if we start adding m2, m3 specific optimizations? |
mh fair enough, I guess it should be clear enough that a m1 optimized version will/should also run on all m series chips |
README.md states "a file with the .nnue extension, storing the neural network for the NNUE evaluation. Binary distributions will have this file embedded.", however my understanding is that Stockfish currently includes a secondary neural network, thus two files. Also, as spelling change suggestions: the CPW refers to itself as "Chess Programming Wiki" and the big endian architecture is spelled "PowerPC". Should a separate PR be created for this? If at all, that is. |
I've added the help and license to https://github.com/official-stockfish/Stockfish/wiki/UCI-&-Commands#help. |
Describe the issue
help
andlicense
commands and command line options (--help
,--license
)Expected behavior
fix above todos.
Steps to reproduce
N/A
Anything else?
N/A
Operating system
All
Stockfish version
master
The text was updated successfully, but these errors were encountered: