-
Notifications
You must be signed in to change notification settings - Fork 4
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
Wdunn001/gpu #1
base: gpu
Are you sure you want to change the base?
Wdunn001/gpu #1
Conversation
Signed-off-by: Jason Dunn <[email protected]>
resolved this issue now receiving two issues with piper and openwakeword Piper:
OpenWakeWord:
|
Do I need a PyTorch instance to run piper on gpu? |
About piper, see the issue I created here: rhasspy/rhasspy3#49 |
@@ -32,13 +32,13 @@ docker compose -f docker-compose.base.yml build --no-cache | |||
Run it with: | |||
|
|||
``` sh | |||
docker composer -f docker-compose.base.yml up -d | |||
docker compose -f docker-compose.base.yml up -d |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for that
@@ -1,3 +1,2 @@ | |||
#!/usr/bin/env bash | |||
python3 -m wyoming_openwakeword \ | |||
--uri 'tcp://0.0.0.0:10400' "$@" | |||
python3 -m wyoming_openwakeword --uri 'tcp://0.0.0.0:10400' "$@" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you change it to single line? Seems unrelated and reduces legibility
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh I did this because if I don't it won't run the script. its not recognizing the / I think this is a different in my environment refactoring it ensures its environment agnostic. I get a command not recognized error for each new line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Weird, this is supposed to run inside the container, should not care about your environment.
No, according to the documentation, you just need |
I think preloading the model should fix the opwnwakeword issue. Not sure if we should preload that one, feels the more neutral and easy to pronounce to me, but maybe I'm wrong. Seems like the maintainer was using ok_nabu, but because 'Nabu Casa' is the company where he works, personally I think the dafault should be something neutral and non comercial word, then they can use it for their products with whatever model they want. |
I just wanted to update you I cannot get OpenWakeWord going I tried preloading a model but they can't be found. I need to redo the docker file so I can peak into the default folder to see if the model is actually there. What I think might need to happen is we allow the end user to define the model location and provide a ling for download to a list of default models. I haven't had time to set all that up recently. |
48f4a98
to
fe4ddc4
Compare
b462ddb
to
f5ef805
Compare
adding commands from non gpu version