-
Notifications
You must be signed in to change notification settings - Fork 486
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
Text2Speech Plugin #4622 #4623
Text2Speech Plugin #4622 #4623
Conversation
Remove the .DS_Store files |
@AngelFQC acabo de hacer los cambios que solicitaste, gracias por el code review! |
Ya entendí, haces el proceso de TTS al momento de la creación del learning path por el plugin AI Helper. Está bien, pero sería mucho mejor si pudieras ofrecer la posibilidad a alguien de hacerlo luego de la generación. En particular en lp_controller.php?action=add_audio, que es una página dedicada al audio y tiene varias secciones de fuentes distintas. El problema de hacerlo únicamente al momento de la generación de la lección es que todavía consideramos que AI Helper solo hará una pre-generación que tiene que ser validada/corregida por el profesor. Por eso, generar el audio sin posibilidad de volver a hacer el proceso es un problema, porque probablemente 50% del tiempo el profesor va a cambiar ligeramente el texto antes de validar (y nos quedaríamos con un audio que no corresponde). Puedes encontrar el formulario normal de add_audio aquí: main/lp/lp_add_audio.php y introducir un check sobre la existencia y activación del plugin text2speech antes de mostrar una sección más para la generación de audio. Observo que no veo de ningún lado una elección de idiomas. Supongo que por ahora quedará así (entiendo que se necesita otro modelo y probablemente otra instancia de servidor TTS para generar el audio en otro idioma), pero hay oportunidades de mejora por este lado (por ejemplo permitiendo la configuración de distintas URLs y API keys según un idioma). |
Quedan dos archivos .DS_STORE, todavía ;-) |
Sugeencia de README.md (que cambiará si agregas el formulario):
|
Other suggestion: Add this to lang/english.php:
spanish.php
french.php
|
$resource = fopen($filePath, 'w'); | ||
|
||
$client = new GuzzleHttp\Client(); | ||
$client->get($this->url.'?api_key='.urlencode($this->apiKey). |
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.
El api_key es necesario? Cómo lo genero? En mi TTS que tengo no pide api_key y me genera los audios por el api solo con el texto
🤔
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.
Creo que no es necesario y que lo pone solo para abrirlo a la posibilidad de tener uno a futuro (para este u otro servicio)
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.
exacto en este momento no es necesario pero lo dejo abierto por si se necesita en algún otro servicio.
Si por lo último que comentas en el caso de mozilla TTS se despliega un modelo por cada idioma, se podría poner algo delante para ir al idioma que se necesite por este momento podemos dejarlo así y ya despues agregar el soporte para multiidioma lo cual creo que es va a tener que ser algo manual, ya que por ejemplo en el LP tendrían que seleccionar el idioma en el que quieren el audio. |
Es más fácil que esto : podemos usar el idioma del curso en el cual el LP está ubicado. Esta parte no me preocupa. Es más saber si hay un servidor TTS que soporte varios modelos a la vez o si se van a tener que lanzar varios servidores TTS, uno para cada modelo/idioma. Pero bueno, iremos descubriendo esto con el tiempo |
Closing this PR. See notes in #4629 for explicit reasons. |
This doesn't diminish our gratitude for your contribution to Chamilo (as the PR based on yours will remain open for a possible later integration) |
This closes #4622