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

Install CPU-optimized version of Stockfish #32

Merged
merged 1 commit into from
Aug 27, 2023
Merged

Conversation

fitztrev
Copy link
Owner

Closes #24

@fitztrev fitztrev merged commit e3f4c82 into main Aug 27, 2023
@fitztrev fitztrev deleted the cpu-architecture branch August 27, 2023 14:02
@dav1312
Copy link
Contributor

dav1312 commented Aug 27, 2023

Note that "modern" is technically deprecated
official-stockfish/Stockfish@d70a905

@fitztrev
Copy link
Owner Author

@dav1312 Thank you!

I noticed that the dev release of Stockfish is back to having other types (bmi2, vvni256, etc) in the Assets but the Stockfish 16 release just had avx2 and modern. Is that how it will be going forward? If so, I may just make those changes and have the app download the dev release.

@vondele
Copy link

vondele commented Aug 27, 2023

In the future, yes, it will be renamed. However, I think the app should download a release and not a prerelease, just in case we introduce a bug in the development, the users of this app shouldn't notice that. Safer to stay with released binaries, IMO.

BTW, would it be meaningful for such an app to verify also the sha256sum of binaries it downloads, just to ensure the binary payload is the expected thing.

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.

Install cpu-optimized version of Stockfish
3 participants