search

CovidTrackerFr / vitemadose-front

Interface utilisateur de l'outil ViteMaDose
https://vitemadose.covidtracker.fr
Other
80 stars 51 forks source link

Calcul Distances Paris #197

Closed Noezor closed 3 years ago

Noezor commented 3 years ago

Peu importe l'arrondissement choisi, c'est toujours les mêmes distances aux centres qui sont calculées. J'ai regardé et ça a l'air de coller avec Notre-Dame. Il faudrait mettre 1 centre par arrondissement pour les grandes villes.

Noezor commented 3 years ago

De ce que j'ai compris, le calcul de distances se fait ici. 2 Hypothèses : soit this.communeSelectionnee porte juste la commune, et pas l'arrondissement. Soit la variable contient bien l’arrondissement, et c'est la résolution de sa location qui se fait mal.

Si quelqu'un est intéressé

bilelz commented 3 years ago

Hello, Dans cette PR #145 , j'avais identifié que l'identifiant unique d'une commune n'est pas le code mais ${this.communeSelectionnee.code}-${this.communeSelectionnee.codePostal}

Ça peut peut-être aider.

fcamblor commented 3 years ago

En fait le problème c'est qu'à la base, les geopositions qu'on a récupéré depuis l'API geoportail partagent toutes les mêmes latng (le "code commune" est le même ... il correspond juste à plusieurs "aliases" de code postaux)

Vaccination_COVID-19_à_Paris_75001
fcamblor commented 3 years ago

Du coup soit on override ce comportement (ce serait possible ... on va notamment sûrement commencer à le faire pour les collectivités d'outremer)

fcamblor commented 3 years ago

Du coup soit on override ce comportement (ce serait possible ... on va notamment sûrement commencer à le faire pour les collectivités d'outremer)

Je crois que j'ai pas fini ma phrase :D

Je pense que l'idée c'était de soit overrider manuellement la latng de certains code postaux, soit de laisser tomber :)