Skip to content

Change 2mic local wakeword docs to use microwakeword #286

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

tannisroot
Copy link
Contributor

I considered leaving instructions for openwakeword too, but microwakeword is both faster and way more accurate for me, plus it would needlessly over-complicate instructions.

@bakhtiyarneyman
Copy link

From upstream README

microWakeword is currently available as an early release. Training new models is intended for advanced users. Training a model that works well is still very difficult, as it typically requires experimentation with hyperparameters and sample generation settings. Please share any insights you find for training a good model!

I enjoy a custom wakeword, so I don't think the information about openWakeWord should be erased at this stage.

@tannisroot
Copy link
Contributor Author

From upstream README

microWakeword is currently available as an early release. Training new models is intended for advanced users. Training a model that works well is still very difficult, as it typically requires experimentation with hyperparameters and sample generation settings. Please share any insights you find for training a good model!

I enjoy a custom wakeword, so I don't think the information about openWakeWord should be erased at this stage.

I can write a shorter version of the instructions at the end saying that the process is pretty much identical except for needing to change the name and okay_nabu to ok_nabu, would that be sufficient?

@bakhtiyarneyman
Copy link

For me? Yes. But I'm not anyhow affiliated with this project. I'm just a user.

@tannisroot
Copy link
Contributor Author

For me? Yes. But I'm not anyhow affiliated with this project. I'm just a user.

Yes, I am aware, input from anyone is important really.

@tannisroot
Copy link
Contributor Author

I've verified the instructions are accurate recently when I had to re-install OS on my zero 2w speaker, though had to add a python3-dev dependency.

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