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

Asterisk Output and Input should assume MRCP #203

Open
5 tasks
benlangfeld opened this issue Jan 12, 2014 · 3 comments
Open
5 tasks

Asterisk Output and Input should assume MRCP #203

benlangfeld opened this issue Jan 12, 2014 · 3 comments

Comments

@benlangfeld
Copy link
Member

Motivation

These days app_swift is utterly unstable and mostly useless. MRCP is used in the vast majority of TTS and ASR deployments, and is the industry standard interface to a TTS or ASR engine.

Additionally, is is sometimes preferable for an application to be more specific about the renderer/recognizer it wishes to invoke by specifying a particular MRCP client profile. This is currently not possible when the only distinction supported is native|unimrcp|swift.

Changes required

  • Remove support for app_swift - this is superseded by the UniMRCP Server Cepstral plugin.
  • renderer/recognizer attributes should specify the UniMRCP profile.
  • Native output rendering with fallback to UniMRCP (default profile) should be the default when no renderer is supplied.
  • Native DTMF detection should be the default when only DTMF grammars are provided and no recognizer is supplied. If speech grammars are supplied, should fall back to the default UniMRCP profile.
  • If UniMRCP is not available, or if rendering or recognition is not possible, a sensible error should be returned.

Application impact

Applications will no longer be able to make use of app_swift. This will necessitate a bump to the major version of Punchblock.

@benlangfeld
Copy link
Member Author

@bklang Comments appreciated.

@bklang
Copy link
Member

bklang commented Jan 17, 2014

While I am a little disappointed to lose app_swift support, I think this is the right direction. I agree that MRCP is the priority interface to support, especially since Cepstral can be reached via MRCP as well.

Can app_swift functionality be moved to a plugin? It may or may not work, but it would allow others who do use app_swift to contribute to its maintenance more easily.

@benlangfeld
Copy link
Member Author

Punchblock doesn't have plugins, it would be more trouble than its worth to try to support that. We could perhaps move it into adhearsion-asterisk, but I would think we'd be better served by killing it completely and documenting the setup with UniMRCP Server.

@benlangfeld benlangfeld modified the milestone: 3.0.0 May 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants