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
Use enum for voiceActivityDetection #467
Conversation
part of #375, based on @martinthomson's [suggestion](#375 (comment))
As we go through the consequences of eliminating true defaults, it gets uglier. Comments wanted. |
If vad is a feature that should be default on because that's what would benefit most people, would it be so bad to have an option called 'disableVoiceActivityDetection'? |
I rather like @adam-be 's suggestion. |
I don't like negative-valued options - "disableVoiceActivityDetection: false" is a double negative. |
<dd>Request voice activity detection to be enabled</dd> | ||
|
||
<dt>disabled</dt> | ||
<dd>Request voice activity detection to be disabled</dd> |
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.
Should we have periods at the end of the sentences?
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.
Otherwise, I'm fine with this.
Closed based on list call-for-consensus results. |
part of #375, based on @martinthomson's suggestion