-
Notifications
You must be signed in to change notification settings - Fork 34
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
[Feature] Send recognition timeout to UniMRCP #228
[Feature] Send recognition timeout to UniMRCP #228
Conversation
This would first require a matching pull request against the Rayo specification. http://xmpp.org/extensions/xep-0327.html#def-component-input |
Could I get test coverage and a changelog entry prior to merging this? Squashing to a single commit would also be awesome :) |
Sorry for the delay; I will get the test and changelog entry in today. |
@benlangfeld this is ready for another look once we hear back from TravisCI |
So you've included tests for the model, but not for the component side. Could you add those too? |
Resolved conflicts: lib/punchblock/translator/asterisk/component/mrcp_recog_prompt.rb send recognition timeout setting as UniMRCP option allowing dynamic setting of Recognition-Timeout in the MRCP messaging with speech recognition server Resolved conflicts: lib/punchblock/translator/asterisk/component/mrcp_recog_prompt.rb add test for setting recognition-timeout header update changelog add spec for mrcp_recog_prompt and enforce Rayo protocol in the #validate method
@@ -34,6 +34,7 @@ def validate | |||
|
|||
raise OptionError, "An initial-timeout value must be -1 or a positive integer." if @initial_timeout < -1 | |||
raise OptionError, "An inter-digit-timeout value must be -1 or a positive integer." if @inter_digit_timeout < -1 | |||
raise OptionError, "A recognition-timeout value must be -1, 0, or a positive integer." if @recognition_timeout < -1 |
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.
added this validation to match the default in the Rayo spec
…eout_to_unimrcp [Feature] Send recognition timeout to UniMRCP
This allows an application to specify recognition timeout settings on a per
#ask
basis when using Asterisk's UniMRCP module for speech recognition.