-
Notifications
You must be signed in to change notification settings - Fork 7
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
Refactorización de código #11
Conversation
…pip install -r requirements.txt. Convertido translator.py a una clase python. AJustado el código para llamar a esa clase.
hola! el código se rompe al intentar invocar a LANGUAGES. supongo que hay que importar a LANGUAGES del translator? Collecting onnxruntime==1.14.1 |
Ciertamente, perdona.
Subo un commit, que se me fue la cabeza y no subí el último, mil disculpas.
Con onnxruntime1.14.1 consigo el siguiente error:
Requirement already satisfied: numpy in c:\users\usuario\appdata\local\programs\python\python311\lib\site-packages (from -r requirements.txt (line 9)) (1.25.2)
ERROR: Could not find a version that satisfies the requirement onnxruntime==1.14.1 (from versions: 1.15.0, 1.15.1)
ERROR: No matching distribution found for onnxruntime==1.14.1
Un saludo.
De: cesar verastegui ***@***.***>
Enviado el: lunes, 4 de septiembre de 2023 3:30
Para: metalalchemist/VeTube ***@***.***>
CC: Jesús Pavón Abián ***@***.***>; Author ***@***.***>
Asunto: Re: [metalalchemist/VeTube] Refactorización de código (PR #11)
hola! el código se rompe al intentar invocar a LANGUAGES. supongo que hay que importar a LANGUAGES del translator?
por otro lado, onnxruntime1.14.1 si existe. no será que por la 1.15.1 se te cuelga el windows? yo lo estoy probando en win10 con el 1.15.1 y se abrió normal. pero es lo que se me ocurrió.
mira, te confirmo.
Collecting onnxruntime==1.14.1
Downloading onnxruntime-1.14.1-cp310-cp310-win_amd64.whl (6.5 MB)
---------------------------------------- 6.5/6.5 MB 6.6 MB/s eta 0:00:00
Requirement already satisfied: coloredlogs in c:\users\cesit\appdata\local\programs\python\python310\lib\site-packages (
from onnxruntime==1.14.1) (15.0.1)
Requirement already satisfied: flatbuffers in c:\users\cesit\appdata\local\programs\python\python310\lib\site-packages (
from onnxruntime==1.14.1) (23.5.26)
Requirement already satisfied: numpy>=1.21.6 in c:\users\cesit\appdata\local\programs\python\python310\lib\site-packages
(from onnxruntime==1.14.1) (1.23.5)
Requirement already satisfied: packaging in c:\users\cesit\appdata\local\programs\python\python310\lib\site-packages (fr
om onnxruntime==1.14.1) (23.1)
Requirement already satisfied: protobuf in c:\users\cesit\appdata\local\programs\python\python310\lib\site-packages (fro
m onnxruntime==1.14.1) (4.21.12)
Requirement already satisfied: sympy in c:\users\cesit\appdata\local\programs\python\python310\lib\site-packages (from o
nnxruntime==1.14.1) (1.12)
Requirement already satisfied: humanfriendly>=9.1 in c:\users\cesit\appdata\local\programs\python\python310\lib\site-pac
kages (from coloredlogs->onnxruntime==1.14.1) (10.0)
Requirement already satisfied: mpmath>=0.19 in c:\users\cesit\appdata\local\programs\python\python310\lib\site-packages
(from sympy->onnxruntime==1.14.1) (1.3.0)
Requirement already satisfied: pyreadline3 in c:\users\cesit\appdata\local\programs\python\python310\lib\site-packages (
from humanfriendly>=9.1->coloredlogs->onnxruntime==1.14.1) (3.4.1)
Installing collected packages: onnxruntime
Successfully installed onnxruntime-1.14.1
—
Reply to this email directly, view it on GitHub <#11 (comment)> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADDAHDSIBFSN22PAKQQ73PDXYUVK7ANCNFSM6AAAAAA4JHLRDE> .
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Efectivamente, no hay una versión de Python 3.11 para onnxruntime 1.14.1. Aun así, la versión actual (1.15) debería correr sin problemas. El paquete se utiliza para controlar los modelos de piper TTS. Se pueden visualizar los wheels de onnxruntime y cualquier paquete disponible en pip aquí |
En efecto, funciona sin problemas. Y si no especificas versión, como hice yo, también lo hace.
De: Mateo Cedillo ***@***.***>
Enviado el: lunes, 4 de septiembre de 2023 14:35
Para: metalalchemist/VeTube ***@***.***>
CC: Jesús Pavón Abián ***@***.***>; Author ***@***.***>
Asunto: Re: [metalalchemist/VeTube] Refactorización de código (PR #11)
Efectivamente, no hay una versión de Python 3.11 para onnxruntime 1.14.1. Aun así, la versión actual (1.15) debería correr sin problemas. El paquete se utiliza para controlar los modelos de piper TTS.
—
Reply to this email directly, view it on GitHub <#11 (comment)> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADDAHDXYO4ZTLRWXRU3TXM3XYXDF3ANCNFSM6AAAAAA4JHLRDE> .
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
sigo recibiendo el mismo error. mira la ruta que tomo es entrar a vetube luego mas opciones luego abrir configuración. Traceback (most recent call last): |
Arreglado, ahora sí.
En efecto, se me había pasado el importar el LANGUAGES.
Ha sido abrir translator.py y darme cuenta.
Acabo de probarlo y ya no se rompe.
De: cesar verastegui ***@***.***>
Enviado el: martes, 5 de septiembre de 2023 2:26
Para: metalalchemist/VeTube ***@***.***>
CC: Jesús Pavón Abián ***@***.***>; Author ***@***.***>
Asunto: Re: [metalalchemist/VeTube] Refactorización de código (PR #11)
sigo recibiendo el mismo error. mira la ruta que tomo es entrar a vetube luego mas opciones luego abrir configuración.
Traceback (most recent call last):
File "D:\descargas\VeTube-traductor\VeTube.py", line 369, in appConfiguracion
self.cf=ajustes.configuracionDialog(self)
File "D:\descargas\VeTube-traductor\ajustes.py", line 34, in init
for k in translator.LANGUAGES: idiomas_disponibles.append(translator.LANGUAGES[k])
File "D:\descargas\VeTube-traductor\translator.py", line 12, in LANGUAGES
return LANGUAGES
NameError: name 'LANGUAGES' is not defined
—
Reply to this email directly, view it on GitHub <#11 (comment)> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADDAHDX3I52KZBJ4EWRDRBTXYZWRZANCNFSM6AAAAAA4JHLRDE> .
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
muchas gracias, lo pruebo y te cuento. por cierto, dejaremos entonces onxruntime asi como lo tienes tu. |
Puede dejarse y mirarlo con detalle en otra pr isue o lo que sea.
… El 5 sept 2023, a las 19:07, cesar verastegui ***@***.***> escribió:
muchas gracias, lo pruebo y te cuento. por cierto, dejaremos entonces onxruntime asi como lo tienes tu.
—
Reply to this email directly, view it on GitHub <#11 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADDAHDUDNWC2S2SPXTPVRB3XY5L5JANCNFSM6AAAAAA4JHLRDE>.
You are receiving this because you authored the thread.
|
Modificado requirements.txt. No dejaba hacer pip install -r requirements.txt por un error en una versión de un módulo.
Convertido translator.py a una clase python.
Ajustado el código para llamar a esa clase.