search

loretoparisi / fasttext.js

FastText for Node.js
MIT License
192 stars 28 forks source link

Fix win32 platform fasttext executable name #28

Closed woozyking closed 2 years ago

woozyking commented 2 years ago

This should fix https://github.com/loretoparisi/fasttext.js/issues/22

@loretoparisi please take a look and let me know if there's any other things needed

tests were conducted preliminarily through running node .\examples\test.js, and the direct invocation of the Util.GetBinFolder function:

> Util.GetBinFolder('fasttext')
'C:\\Users\\woozyking\\proj\\fasttext.js\\lib\\bin\\win32\\fasttext.exe'

since Windows FS isn't strict about lettercases, althought it should have been strictly fastText.exe, this still works