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

NNUE_EMBEDING_OFF as default #6

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rgen3
Copy link

@rgen3 rgen3 commented Jan 3, 2022

Disables NNUE as default pod behavior

@veloce
Copy link
Owner

veloce commented Jan 5, 2022

Why do you need that?

@rgen3
Copy link
Author

rgen3 commented Jan 11, 2022

Hi, sorry for the short description, let me explain
Setting up this plugin with nnue is not simple thing and unfortunately this plugin is not well documented.

So for me as a newby to capacitor and ios development really difficult to setup a plugin with downloading .nn file and including it to build process. So that, the only way I could install this plugin, was to find this constant and turn building with nnue off.

This PR I've opened as an example for other people, how to install and try stockfish in mobile development without painful installation of nn.

Maybe I should open an issue with documentation label and put this hack there

@veloce
Copy link
Owner

veloce commented Mar 24, 2022

Sorry if I have not responded earlier. I agree that documentation is lacking here on how to embed the nn file, and it's not easy to find. I'll consider the default NNUE_EMBEDING_OFF but it should be done with a proper documentation and a warning of what are the differences when using or not NNUE.
So I'm creating an issue to tackle the documentation first.

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