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

Bug: Node version - npm install, empty index.js in node_modules/stt #1830

Closed
pietrop opened this issue Apr 8, 2021 · 11 comments
Closed

Bug: Node version - npm install, empty index.js in node_modules/stt #1830

pietrop opened this issue Apr 8, 2021 · 11 comments
Labels
bug Something isn't working

Comments

@pietrop
Copy link

pietrop commented Apr 8, 2021

Describe the bug
In the Node version when installing via npm install stt (v 0.10.0-alpha.4) and running the code example, you get an error

TypeError: Ds.Model is not a constructor

But most importantly if you look inside node_modules/stt/index.js the file is empty.

To Reproduce

  1. create a new node project with npm init
  2. do npm install stt
  3. inspect node_modules/stt/index.js
  4. See error - index.js is empty

Expected behavior
To be able to install stt via npm, import/require the module and use it etc..

Environment (please complete the following information):
Mac OS X

Additional context
Where's the code that is installed via npm? is it in one of the folders in this repo https://github.com/coqui-ai/STT?

@pietrop pietrop added the bug Something isn't working label Apr 8, 2021
@reuben
Copy link
Collaborator

reuben commented Apr 8, 2021

We currently only have published pre-built packages for Python on Linux and Python on macOS. We're working as quickly as possible to get other platforms and languages up and running (via porting the CI to GitHub Actions). The empty packaged was pushed to reserve the package name.

@reuben
Copy link
Collaborator

reuben commented Apr 8, 2021

In the mean time you can still build it from source.

@pietrop
Copy link
Author

pietrop commented Apr 9, 2021

Thanks, I'll wait for the npm version to be available. And let me know if there's anything I could help with.

@johanneskropf
Copy link

johanneskropf commented Apr 13, 2021

This is good to know, I already ported my deepspeech integration for node-red to coqui-stt as I will focus on that in the future but will wait publishing it till the npm package works.

best regards from Berlin, Johannes

@reuben
Copy link
Collaborator

reuben commented Apr 28, 2021

I've just published version 0.10.0-alpha.5 of STT and STT-tflite on npmjs.com. So you should now be able to use the pre-built NodeJS/ElectronJS bindings with a simple npm install instead of building from source. Thanks for your patience!

@reuben reuben closed this as completed Apr 28, 2021
@johanneskropf
Copy link

johanneskropf commented Apr 28, 2021

Im getting this error when I try to use it:

Error: Cannot find module '/home/pi/node-red-contrib-coqui-stt/node_modules/stt/lib/binding/v0.10.0-alpha.5/linux-arm/node-v72/stt.node'

So I checked and there is no arm support yet in the npm package. Is there a plan to add Linux arm support like deepspeech has?

@reuben
Copy link
Collaborator

reuben commented Apr 28, 2021

Yep, coming soon. I should have specified but I didn't realize you were looking for ARM support :)

@johanneskropf
Copy link

That is good to know as most people using my nodered deepspeech node now I know of do so on one kind of single board computer or another as that is the most popular hardware choice for nodered. So this will be the prerequisite for me to move on to focus on and publish the coqui node for nodered.
Thanks for the continued effort you girls and guys put into the project 👍😀

@johanneskropf
Copy link

johanneskropf commented May 25, 2021

Just to catch up on this @reuben is there an eta on the arrival of Linux arm64 or armhf support of the npm binding? I would really like to focus my development on coqui instead of deepspeech but can't as long as it's not available for the majority of my users unfortunately 🙁
Should I open a new issue for this?

@reuben
Copy link
Collaborator

reuben commented May 28, 2021

@johanneskropf hopefully by EOD today :)

@reuben
Copy link
Collaborator

reuben commented May 28, 2021

@johanneskropf I think I got confused somehow. stt-tflite 0.10.0-alpha.6 should already have ARMv7 and ARMv8 support in it. Are you sure you were trying stt-tflite and not stt?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants