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

add universal model for Neo Ya #465

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

add universal model for Neo Ya #465

wants to merge 3 commits into from

Conversation

eqiihuu
Copy link

@eqiihuu eqiihuu commented Jul 4, 2018

Add a universal model for the keyword "Neo Ya". There are 4 models in the .umdl file, and the suggested sensitivities are [0.6, 0.6, 0.6, 0.6].

Copy link
Collaborator

@chenguoguo chenguoguo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check the comments. Also, does the model compatible with the current libraries on github?

README.md Outdated
* **resources/models/smart_mirror.umdl**: Universal model for the hotword "Smart Mirror" (https://snowboy.kitt.ai/hotword/47). Set sensitivity to Sensitivity to 0.5, and ApplyFrontend to false.
* **resources/models/neo_ya.umdl**: Universal model for the hotword "Neo Ya" (https://snowboy.kitt.ai/hotword/22171). It has four different models for the hotword "Neo Ya", so you have to use two sensitivites "0.6, 0.6, 0.6, 0.6", and ApplyFrontend to true.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No space between sensitivities.

README.md Outdated
@@ -165,8 +165,9 @@ Here is the list of the models, and the parameters that you have to use for them

* **resources/alexa/alexa-avs-sample-app/alexa.umdl**: Universal model for the hotword "Alexa" optimized for [Alexa AVS sample app](https://github.com/alexa/alexa-avs-sample-app). Set SetSensitivity to 0.6, and set ApplyFrontend to true. This is so far the best "Alexa" model we released publicly, when ApplyFrontend is set to true.
* **resources/models/snowboy.umdl**: Universal model for the hotword "Snowboy". Set SetSensitivity to 0.5 and ApplyFrontend to false.
* **resources/models/jarvis.umdl**: Universal model for the hotword "Jarvis" (https://snowboy.kitt.ai/hotword/29). It has two different models for the hotword Jarvis, so you have to use two sensitivites. Set sensitivities to "0.8,0.80" and ApplyFrontend to true.
* **resources/models/jarvis.umdl**: Universal model for the hotword "Jarvis" (https://snowboy.kitt.ai/hotword/29). It has two different models for the hotword Jarvis, so you have to use two sensitivites. Set sensitivities to "0.8, 0.8" and ApplyFrontend to true.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove the space between the numbers, only use ",". The reason is that we interpret the sensitivities as a string in C++, and we use "," as the delimiter.

README_ZH_CN.md Outdated
@@ -158,6 +158,7 @@ Snowboy底层库由C++写成,通过swig被封装成能在多种操作系统和
* **resources/models/snowboy.umdl**:唤醒词为“snowboy”的通用模型。将`SetSensitivity`设置为`0.5`,`ApplyFrontend`设置为`false`。
* **resources/models/jarvis.umdl**: 唤醒词为“Jarvis” (https://snowboy.kitt.ai/hotword/29)的通用模型,其中包含了对应于“Jarvis”的两个唤醒词模型,所以需要设置两个`sensitivity`。将`SetSensitivity`设置为`0.8,0.8`,`ApplyFrontend`设置为`true`。
* **resources/models/smart_mirror.umdl**: 唤醒词为“Smart Mirror” (https://snowboy.kitt.ai/hotword/47)的通用模型。将`SetSensitivity`设置为`0.5`,`ApplyFrontend`设置为`false`。
* **resources/models/neo_ya.umdl**: 唤醒词为"Neo Ya" (https://snowboy.kitt.ai/hotword/22171)的通用模型,其中包含了对应于"Neo Ya"的四个唤醒词模型, 所以需要设置四个`sensitivites`。将`SetSensitivity`设置为`0.6, 0.6, 0.6, 0.6`,`ApplyFrontend`设置为`true`。
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No space between sensitivities.

@eqiihuu
Copy link
Author

eqiihuu commented Jul 5, 2018

I have removed the whitespace between sensitivities. Besides, the model is compatible with the current libraries

README.md Outdated
* **resources/models/smart_mirror.umdl**: Universal model for the hotword "Smart Mirror" (https://snowboy.kitt.ai/hotword/47). Set sensitivity to Sensitivity to 0.5, and ApplyFrontend to false.
* **resources/models/neo_ya.umdl**: Universal model for the hotword "Neo Ya" (https://snowboy.kitt.ai/hotword/22171). It has four different models for the hotword "Neo Ya", so you have to use two sensitivites "0.6,0.6,0.6,0.6", and ApplyFrontend to true.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"so you have to use two sensitivities" --> "so you have to use four sensitivities"

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