[DO NOT MERGE] Consolidation IRVE : une seule passe de correction des coordonnées#5560
Open
vdegove wants to merge 2 commits into
Open
[DO NOT MERGE] Consolidation IRVE : une seule passe de correction des coordonnées#5560vdegove wants to merge 2 commits into
vdegove wants to merge 2 commits into
Conversation
vdegove
changed the title
vdegove
marked this pull request as ready for review
July 13, 2026 10:34
vdegove
added
the
IRVE
github-project-automation
Bot
moved this to Planned
in Consolidation IRVE transport.data.gouv
vdegove
moved this from Planned
to In progress
in Consolidation IRVE transport.data.gouv
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
PR petite sœur de #5559, à merger après #5559. Je l’extrait dans une PR à part pour que #5559 reste digeste. Le diff ce cette PR indique la différence envers la branche
irve-single-parseet non enversmaster.Après #5559, il reste un seul traitement qui est joué deux fois de suite : celui du process des coordonnées et du calcul de l’inversion de celles-ci. On le fait lors de la validation (pour émettre le warning), puis lors de l’insertion en base (pour avoir des coordonnées en base correctes).
Cette PR passe à un seul traitement.
Conséquence importante (et choix d’architecture, au passage) : le DataFrame en sortie du validateur contient des données rectifiées, puisque on fait en une seule passe la détection et l’émission du warning (ah tiens les coordonnées sont inversées) et l’inversion proprement dite.
À vrai dire pas entièrement serein là dessus, c’est à discuter, on peut aussi faire en sorte que le validateur ne fasse qu’émettre le warning, avec des nouvelles colonnes latitude et longitude non rectifiées, et que ça soit le module de préparation pour l’insertion en base qui s’occupe de les swapper, mais c’est un peu artificiel. À voir.