Exemple Millau | Estimation rayon et distance interne
Nouhaila13 opened this issue · 2 comments
Nouhaila13 commented
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
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
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 !