Exemple Millau | Estimation rayon et distance interne

mobility-team / mobility

Mobility, an open-source library for mobility modelisation

MIT License

16 stars 10 forks source link

Exemple Millau | Estimation rayon et distance interne #38

Open Nouhaila13 opened 1 year ago

Nouhaila13 commented 1 year ago

Bonjour, Dans le code de l'exemple Millau, To do: compute the internal distance within the code instead of using a CSV

Après avoir passé en revue les calculs de la distance interne et le rayon dans le fichier Excel donnéescommunefrance, c'est plutôt R= sqrt(surface/pi) et non pas R= sqrt(surface)/pi. Étant donnée l'importance de ce paramètre notamment dans le calcul de la distance interne qui vont nous servir à la réalisation de la tache.

Mind-the-Cap commented 1 year ago

Bonjour @Nouhaila13,

En effet, la bonne formule est R = sqrt(surface/pi), bien vu.

Pour l'instant, le calcul est fait dans le CSV, pas dans le code, c'est justement à faire. Avez-vous pu vérifier si le calcul fait dans le CSV était le bon ?

Pour corriger ça, vous pouvez proposer une PR :

soit en corrigeant le commentaire en cause ;
soit (encore mieux) en implémentant le calcul dans le code.

Merci !

Nouhaila13 commented 1 year ago

Bonjour @Mind-the-Cap ,

En effet, on a vérifié le calcul dans le CSV aussi, les résultats pour le rayon et la distance interne sont aussi calculés par la formule erronée.
Je vais proposer une PR pour travailler sur le deuxième point.

Merci !