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

Upgrade Python dependencies #316

Closed
wants to merge 4 commits into from

Conversation

why-not-try-calmer
Copy link
Contributor

No description provided.

@why-not-try-calmer why-not-try-calmer force-pushed the 301-upgrade-python-dependencies branch 2 times, most recently from 2518ec2 to c2a40df Compare December 19, 2023 15:54
@why-not-try-calmer
Copy link
Contributor Author

A noter que cette PR met à jour la documentation en offrant une méthode fiable pour mettre à jour les dépendances.

@spch-GL
Copy link
Contributor

spch-GL commented Jan 9, 2024

Merci, je me réjouis d'avance

@why-not-try-calmer why-not-try-calmer modified the milestone: Finish p3 Jan 9, 2024
@spch-GL
Copy link
Contributor

spch-GL commented Jan 10, 2024

Remarques:

  1. django :
  • 5.0 nécessite python 3.10 or nous avons 3.9
  • 4.2 nécessite PostgreSQL 12 or nous avons 11.5 en DEV, 15.0 en test, et 14.1 en prod
  • Je suis bloqué sur 3.2.23 pour l'instant. Mais je vais voir pour upgrader...
  1. nose2 ;
  • Pas installé sur mon poste, est-il vraiment nécessaire ?
  1. plotly :

@why-not-try-calmer
Copy link
Contributor Author

why-not-try-calmer commented Jan 10, 2024

  • 5.0 nécessite python 3.10 or nous avons 3.9

ajusté

  • 4.2 nécessite PostgreSQL 12 or nous avons 11.5 en DEV, 15.0 en test, et 14.1 en prod

... nécessite au moins PostgreSQL 12. C'est compliqué ou non faisable de mettre à jour votre environnement dev de 11.5 -> 15? En tout cas j'ai ajouté une matrice de tests à la CI pour que vous puissiez suivre l'état de la compatibilité du plugin ave les versions 12 et 15 de Postgis.

  • Je suis bloqué sur 3.2.23 pour l'instant. Mais je vais voir pour upgrader...

non affecté par cette PR

  1. nose2 ;
  • Pas installé sur mon poste, est-il vraiment nécessaire ?

pas vraiment, c'est utilisé pour quelques tests, je préfère laisser ça comme je l'ai trouvé si ça ne vous embête pas.

  1. plotly :

ajusté

test matrix for postgis versions
@spch-GL
Copy link
Contributor

spch-GL commented Jan 11, 2024

  • Je suis bloqué sur 3.2.23 pour l'instant. Mais je vais voir pour upgrader...

Désolé, je n'ai pas été très clair : Je voulais dire que j'allais changer la version de PG dans mon environnement de DEV. Cela sera la 15.5 (avec PostGIS 3.4.1) à moins que tu n'aie des contre-indications...

@why-not-try-calmer
Copy link
Contributor Author

  • Je suis bloqué sur 3.2.23 pour l'instant. Mais je vais voir pour upgrader...

Désolé, je n'ai pas été très clair : Je voulais dire que j'allais changer la version de PG dans mon environnement de DEV. Cela sera la 15.5 (avec PostGIS 3.4.1) à moins que tu n'aie des contre-indications...

Nickel, juste pour être clair aucun test ici présent ne s'occupe de PostgreSQL "pur", tous concernent PostGIS (PostGresSQL + les fonctions GIS, le tout sous un autre "emballage").

@spch-GL
Copy link
Contributor

spch-GL commented Jan 17, 2024

Mon environnement de DEV est maintenant avec PG 15.5. Django 4.2.9 est fonctionnel.

@why-not-try-calmer
Copy link
Contributor Author

Super, merci!

@spch-GL
Copy link
Contributor

spch-GL commented Jan 17, 2024

Lorsque je lance pip-compile --upgrade il y a un message d'erreur:
image

En plus, ce 'C:\apps\OSGeo4W\bin\python39.zip' n’existe pas…
Par contre, l'exe est bien là 'C:\apps\OSGeo4W\bin\python39.exe'

J'ai installé pip-tools 7.3.0 et je n'ai pas trouvé d'indication qu'il ne fonctionnerait pas avec python 3.9

Par contre je n'ai pas de venv, est-ce que cela pourrait être la source du problème ?
Ou faut-il aussi installer 'encodings' ? Mais alors, comment ?

17.01.2024_11:40:17.90 C:\apps\OSGeo4W>pip install encodings
ERROR: Could not find a version that satisfies the requirement encodings (from versions: none)
ERROR: No matching distribution found for encodings

17.01.2024_11:45:40.97 C:\apps\OSGeo4W>

@why-not-try-calmer
Copy link
Contributor Author

why-not-try-calmer commented Jan 17, 2024

Je ne connais pas les détails de l'installation de Python sur Windows. La documentation concernant la mise à jour des dépendances (après cette PR, puisque celle-ci fait déjà une mise à jour) est d'ailleurs adressée aux développeurs qui prendront la relève, et qui a coup sûr auront une machine non-Windows à disposition. Si toutefois -- et c'est une bonne chose ! -- tu souhaites toi-même tester la procédure de mise-à-jour le mieux à faire est probablement d'utiliser Docker?

Je te donne volontiers un coup de main pour la mise en route mais ça m'éloigne pas mal du sujet principal (p1-p3), peut-être à voir lors du meeting? (Quand d'ailleurs?)

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

Successfully merging this pull request may close these issues.

Quels principes utiliser concernant les évolutions des composants ?
2 participants