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

Refactorización de código #11

Merged
merged 3 commits into from
Sep 5, 2023

Conversation

jpavonabian
Copy link
Contributor

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.

…pip install -r requirements.txt. Convertido translator.py a una clase python. AJustado el código para llamar a esa clase.
@metalalchemist
Copy link
Owner

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

@jpavonabian
Copy link
Contributor Author

jpavonabian commented Sep 4, 2023 via email

@rmcpantoja
Copy link
Collaborator

rmcpantoja commented Sep 4, 2023

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í

@jpavonabian
Copy link
Contributor Author

jpavonabian commented Sep 4, 2023 via email

@metalalchemist
Copy link
Owner

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

@jpavonabian
Copy link
Contributor Author

jpavonabian commented Sep 5, 2023 via email

@metalalchemist
Copy link
Owner

muchas gracias, lo pruebo y te cuento. por cierto, dejaremos entonces onxruntime asi como lo tienes tu.

@metalalchemist metalalchemist merged commit fda362f into metalalchemist:master Sep 5, 2023
@jpavonabian
Copy link
Contributor Author

jpavonabian commented Sep 5, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants