Fix requirements #70
Open
Fix requirements #70
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
la última versión de python_telegram_bot que tiene es la 13.5 https://docs.python-telegram-bot.org/en/v13.15/telegram.chataction.html vs 404: https://docs.python-telegram-bot.org/en/v20.0a0/telegram.chataction.html parece haber sido movida a: https://docs.python-telegram-bot.org/en/latest/telegram.constants.html#telegram.constants.ChatAction pero actualizar requiere más tiempo
si se corre con python3.11 aparece este error: Unsupported operation: RETURN_GENERATOR ... el problema es el ORM pony: https://stackoverflow.com/questions/75907458/unsupported-operation-return-generator-on-select#comment133888098_75907458 parece que no soporta python3.11 se modifica el readme para dar indicaciones utiles
iglosiggio
reviewed
Jul 15, 2023
Comment on lines
+32
to
+44
| Otra es usar **guix** que permite además crearte todo el ambiente y tener una shell contenerizada, para instalarlo podés ver el primer link de [la guia de instalación binaria](https://guix.gnu.org/manual/en/html_node/Binary-Installation.html) que apunta a un script que automatiza el proceso. | ||
| Luego: | ||
| ```bash | ||
| guix shell python3.10.7 | ||
| ``` | ||
| y luego seguir con la instalación del ambiente virtual. | ||
|
|
||
| también podés correr dicha shell en un container, pero en ese caso tenés que instalar más paquetes: | ||
| ```bash | ||
| guix shell -CN coreutils tzdata [email protected] | ||
| ``` | ||
| '-C' crea el container y '-N' habilita el uso de la red. | ||
| tené en cuenta que vas a tener que recrear el ambiente virtual en estes caso. |
There was a problem hiding this comment.
Suggested change
| Otra es usar **guix** que permite además crearte todo el ambiente y tener una shell contenerizada, para instalarlo podés ver el primer link de [la guia de instalación binaria](https://guix.gnu.org/manual/en/html_node/Binary-Installation.html) que apunta a un script que automatiza el proceso. | |
| Luego: | |
| ```bash | |
| guix shell python3.10.7 | |
| ``` | |
| y luego seguir con la instalación del ambiente virtual. | |
| también podés correr dicha shell en un container, pero en ese caso tenés que instalar más paquetes: | |
| ```bash | |
| guix shell -CN coreutils tzdata [email protected] | |
| ``` | |
| '-C' crea el container y '-N' habilita el uso de la red. | |
| tené en cuenta que vas a tener que recrear el ambiente virtual en estes caso. |
Banco que guix está piola pero no es el lugar para hacerle publicidad imho
| @@ -1,5 +1,5 @@ | |||
| pony | |||
| python_telegram_bot | |||
| python_telegram_bot==13.5 | |||
There was a problem hiding this comment.
El pin de versión es importante?
| @@ -6,7 +6,41 @@ Mientras seguimos la migración todos son libres de colaborar con pull requests. | |||
|
|
|||
| Para probar el bot en acción en Telegram es necesario crear un archivo `tokenz.py` que asigne a la variable `token` el token del bot con el que se desea hacer las pruebas. | |||
|
|
|||
| Para instalar las dependecias correr lo siguiente: | |||
| Para instalar las dependencias correr lo siguiente: | |||
Comment on lines
+16
to
+17
| $ python3 -m venv venv | ||
| $ venv/bin/pip3 install -r requirements.txt |
There was a problem hiding this comment.
No va un source venv/bin/activate (o activate.zsh o activate.fish) en el medio?
Comment on lines
+20
to
+31
| para que el bot tenga los comandos hay que instalarlos en la base de datos: | ||
| ```bash | ||
| $ venv/bin/python3 install.py | ||
| ``` | ||
| y ahora si es posible correr el bot: | ||
| ```bash | ||
| $ venv/bin/python3 dcubabot.py | ||
| ``` | ||
|
|
||
| ## notas: | ||
| - la librería pony no funciona con python3.11, así que vas a necesitar instalarte python3.10 | ||
| tenés muchas alternativas, descargartelo de la web, usar el paquete de tu distro, etc |
There was a problem hiding this comment.
muchas muchas muchas gracias por esto <3
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
tardé un poco más de lo esperado en levantar el bot localmente para ver si los cambios no rompian nada, así que fixee algunas cosas y rellené un poco el readme