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

Executable not found in path on Windows #22

Closed
MarcDelp opened this issue Dec 11, 2019 · 7 comments · Fixed by #28
Closed

Executable not found in path on Windows #22

MarcDelp opened this issue Dec 11, 2019 · 7 comments · Fixed by #28

Comments

@MarcDelp
Copy link

Hi !

Node version: 12.13.1
OS: Windows 10

I'm trying to create a simple fastText model for text classification, but the lib is not working fine on Windows. As soon as I add this code:

const fastText = new FastText({
      serializeTo: './models/fastText',
      trainFile: './trainingData.txt'
});

I have the error:

Error: executable not found in path
    at new FastText (C:\Users\madelpech\[...]\node_modules\fasttext.js\lib\index.js:104:23)

I have looked to the code and in the index.js file line 99 you are calling this._options.bin = Util.GetBinFolder('fasttext');. I have checked the binaries and it appears that the Windows binary is not named fasttext but fastText.exe instead. This should be the issue here.

Can you have a look and tell me if I am right?

Thanks!

@loretoparisi
Copy link
Owner

@MarcDelp yes code it is ok, but for MS windows the support is out dated.
The win32 executable is provided in /lib/bin/win32 folder and yes just try to change the name to fastText.exe as executable file path. Windows support for 32 bit applications in 64bit OS it's ok, so it should work.

@MarcDelp
Copy link
Author

Okay @loretoparisi
Thanks for the quick response :)

@feernando998
Copy link

Hi !

Node version: 12.13.1
OS: Windows 10

I'm trying to create a simple fastText model for text classification, but the lib is not working fine on Windows. As soon as I add this code:

const fastText = new FastText({
      serializeTo: './models/fastText',
      trainFile: './trainingData.txt'
});

I have the error:

Error: executable not found in path
    at new FastText (C:\Users\madelpech\[...]\node_modules\fasttext.js\lib\index.js:104:23)

I have looked to the code and in the index.js file line 99 you are calling . I have checked the binaries and it appears that the Windows binary is not named but instead. This should be the issue here.this._options.bin = Util.GetBinFolder('fasttext');``fasttext``fastText.exe

Can you have a look and tell me if I am right?

Thanks!

Hello MarcDelp,, were you successful in using fasttext with Js on windows? Thanks.

@MarcDelp
Copy link
Author

Hey @feernando998

Honestly, I can't remember what was my final thought on this... I think that I ended changing the exe name for my local installation since the deployed image was based on Linux and didn't need correction. This enabled me to develop on my local and then I could check on the image built that everything was fine.

@feernando998
Copy link

Hey @feernando998

Honestly, I can't remember what was my final thought on this... I think that I ended changing the exe name for my local installation since the deployed image was based on Linux and didn't need correction. This enabled me to develop on my local and then I could check on the image built that everything was fine.

Got it, thanks for answering me!

@loretoparisi
Copy link
Owner

@feernando998 @MarcDelp thank you guys, feel free to submit a PR to fix this issue, currently I do not have a Windows machine to test it, but if you can submit a PR, so that it will go in in the repo.

Thanks a lot.

@searleser97
Copy link

I am still seeing this issue, Why is it closed?

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 a pull request may close this issue.

4 participants