mobility-team / mobility

Mobility, an open-source library for mobility modelisation
MIT License
16 stars 10 forks source link

Données | Exemple Millau #37

Open AyoubFOUNDOU opened 1 year ago

AyoubFOUNDOU commented 1 year ago

Dans le Todo de get_data_for_model :

Voici le fichier (https://www.data.gouv.fr/fr/datasets/r/dbe8a621-a9c4-4bc3-9cae-be1699c5ff25) qu'on a utilisé pour éviter de travailler avec des données locales, mais on n'a pas eu les mêmes résultats qu'avant.

Mind-the-Cap commented 1 year ago

Bonjour @AyoubFOUNDOU et merci pour l'issue,

C'est une très bonne question, j'ai utilisé un fichier qui était déjà là et que mon prédécesseur n'a pas documenté non plus (honte à nous). Ce serait une très bonne chose de le remplacer par un fichier bien identifié comme vous l'avez tenté. D'autant plus que les cordonnées sont effectivement dans un format inhabituel.

À quel point vos résultats sont-ils différents ? N'hésitez pas à les partager sous une forme ou une autre, et à partager le code que vous avez tenté d'utiliser.

AyoubFOUNDOU commented 1 year ago

Bonjour @Mind-the-Cap,

On a réussi à trouver la méthode qui a été utilisée dans la conversion de latitude et longitude aux coordonnées x,y. En effet. On se base sur la norme en vigueur en France pour les coordonnées cartésiennes : Lambert-93 (C’était la norme standard qui était utilisée dans le fichier csv). Après avoir exécuté ce script, on a constaté qu’il fallait beaucoup de temps pour faire la conversion des coordonnées. Pour cette raison, on propose de garder les données des communes en local pour éviter de refaire la conversion à chaque fois.

Le script qu’on a utilisé pour convertir les coordonnées est accessible par le lien suivant :

https://colab.research.google.com/drive/1RCqwQQN9M1Fkr4OMqpG5w80cxImb9epk?usp=sharing

Mind-the-Cap commented 1 year ago

Merci @AyoubFOUNDOU

Quelle est votre suggestion pour le code de mobility soit meilleur ? Conserver la solution actuelle et améliorer la doc ? Intégrer le script pour être sûr de savoir repasser en Lambert-93 si besoin ? Ou faire en sorte d'utiliser un autre système de coordonnées ?

Selon la solution qui vous paraît la plus adaptée, vous pouvez commencer une PR (assez simple dans le premier cas, plus compliquée dans le dernier). Votre travail est important pour le moment où de nouvelles données arriveront sur les communes (potentiellement tous les ans au gré des fusions).

AyoubFOUNDOU commented 1 year ago

On propose de garder le fichier CSV actuel (pour éviter de refaire les calculs à chaque fois qu’on utilise l’application) avec une correction des distances internes puisqu'elles n'étaient pas calculées correctement. Pour la méthode de conversion des coordonnées on propose de la mettre comme commentaire dans le code. La chose qui reste à voir est comment vous souhaitez faire la mise à jour de ce fichier CSV pour qu’il soit synchronisé avec la base de données de l’INSEE ?

Mind-the-Cap commented 1 year ago

Ok pour moi, merci ! Pour le dernier point, ça veut dire qu'il faut mettre un avertissement dans le code d'aller vérifier ponctuellement les bases de données de l'INSEE + un mode d'emploi pour produire le CSV à partir de ces dernières ; c'est un peu manuel mais à ce niveau de développement du projet ça me semble ok.