Skip to content

Feature/docker compose db#24

Open
alexpetit wants to merge 5 commits intodataforgoodfr:mainfrom
alexpetit:feature/docker-compose-db
Open

Feature/docker compose db#24
alexpetit wants to merge 5 commits intodataforgoodfr:mainfrom
alexpetit:feature/docker-compose-db

Conversation

@alexpetit
Copy link
Copy Markdown
Contributor

Ajout d'une base de données pour la partie géolocalisation. Intégration avec docker compose à venir

@TimCo31
Copy link
Copy Markdown
Collaborator

TimCo31 commented Apr 7, 2026

Hello @alexpetit ,
J'ai fait qques modifs de mon côté pour que mon docker-compose fonctionne. Ca fonctionnait bien de ton côté ?
Mais une fois ces modifs, le flow et l'enrichissement marchent bien, donc c'est cool !
Juste qques trucs que j'ai vu :

  • Plutôt que mettre des print pour suivre les avancements, mieux vaut passer par logger.info pour que ca ressorte dans tes log docker.
  • Sur l'emplacement du dockerfile et du dockerignore. En gros là comme tes fichier sont dans infra, l'app que tu construis correspond pas à ta pipeline j'ai l'impression. Donc dans l'idée moi je le mettrai quand même plutôt ces fichiers à la racine du dossier. En soit ca fonctionnerait peut être aussi de modifier ce qui est dans le dockerignore, mais en l'état ca ne permet pas d'ignorer tes dossiers data, etc. Et donc ca vient surcharger pas mal ton container.
  • Dans postgres.py, j'ai vu que tu avais une fonction pour créer les tables si elles n'existaient pas. Ca peut être une idée, mais je pense que mieux vaut dans tous les cas les créer avec un fichier init.sql en amont pour initialiser tes tables.
  • Pour le split des fichiers toml, j'ai pas vraiment d'avis. Comment est-ce que fonctionne uv dans ce cas là pour sélectionner le bon environnement de travail ?

En vrai je ne suis pas expert de docker donc peut être que ton truc est mieux

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.

2 participants