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

Compiling #21

Closed
SzotsGabor opened this issue Dec 9, 2021 · 3 comments
Closed

Compiling #21

SzotsGabor opened this issue Dec 9, 2021 · 3 comments

Comments

@SzotsGabor
Copy link

Using the recommended command line (cargo build --release --features 'nnue') I get this error message:

error: none of the selected packages contains these features: 'nnue', did you mean: nnue?

I can successfully compile it with the simple cargo build --release but I am not sure the NNUE is included in the exe. Also, I get many warnings (mostly about unused variables).

@jnlt3
Copy link
Owner

jnlt3 commented Dec 9, 2021

In that case, doing just nnue and not 'nnue' should fix the issue.

About the other issue, I'll recompile the binaries and release them without the "dev" their names.

@SzotsGabor
Copy link
Author

In that case, doing just nnue and not 'nnue' should fix the issue.

About the other issue, I'll recompile the binaries and release them without the "dev" their names.

Yes, nnue without the quotes was accepted. Thanks on both accounts.

If I understand things correctly, the nnue.bin file is not needed to run the engine, it is embedded in the executable.

@jnlt3
Copy link
Owner

jnlt3 commented Dec 9, 2021

Yes, it is parsed at compile time and included in the binary. I'll also make sure to not include the quotes in the next versions in order to make things easier.

@jnlt3 jnlt3 closed this as completed Dec 9, 2021
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

No branches or pull requests

2 participants