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

Calcul des codes postaux en utilisant Datanova et les contours postaux #439

Merged
merged 9 commits into from
Jul 8, 2024

Conversation

jbouhadoun
Copy link
Member

@jbouhadoun jbouhadoun commented Jun 27, 2024

Cette pull request améliore la précision des calculs des codes postaux en utilisant des sources de données supplémentaires et fiables, tout en conservant une méthode de secours en cas d'absence de données dans les nouvelles sources.

  • Utilisation des fichiers datanova.csv et contours_postaux.geojson situés dans db-migration/data.
  • Calcul des codes postaux en croisant les données de ces fichiers.
  • Si les codes postaux ne sont pas présents dans les deux fichiers, utilisation de l'ancienne méthode.

Remarque : Les fichiers datanova.csv et contours_postaux.geojson ne sont pas inclus dans le dépôt Git.

Exemple sur Dunkerque

  • Bleu : Le code postal
  • Rouge : Le code postal calculé avant cette PR avec l'ancienne fonction.
  • Vert : Le code postal calculé avec la nouvelle fonction
    image
    image

@jbouhadoun jbouhadoun linked an issue Jun 27, 2024 that may be closed by this pull request
@antoineludeau
Copy link
Member

antoineludeau commented Jul 1, 2024

  • Pour les fichiers datanova.csv et contours_postaux.geojson, il faudrait les mettre sur le S3 et les lire en mémoire en stream pendant la migration (solution idéale). Ou alors dans un premier temps, les ajouter au script de download-dataset pour les télécharger de quelque part (solution non-idéale) avant le lancement des fichiers de migration.

Exemple pour la solution avec lecture en stream sur S3:

const params = {
      Bucket: process.env.S3_BUCKET_NAME,
      Key: 'path/to/your/datafile.json', 
    };

const s3Stream = s3.getObject(params).createReadStream();
  • Pour les vues, on pourrait utiliser les vues déjà existantes sur adresse (address_view) et toponyme commun (common_toponym_view) et ainsi ne pas refaire de query pour récupérer l'info des codes postaux.

@jbouhadoun jbouhadoun self-assigned this Jul 3, 2024
@jbouhadoun jbouhadoun force-pushed the jugurtha/integrer-les-contours-postaux-dans-pg branch from 53ab16f to 71409df Compare July 5, 2024 12:11
@jbouhadoun jbouhadoun merged commit 1997d5a into master Jul 8, 2024
1 check passed
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.

Intégrer les contours postaux dans PG
2 participants