search

fabmob / tracemob-phone

Tracemob is a Mobile App developed by fabmob for la Rochelle's Agremob project (Self mobility Data) in 2021 ; it is a fork from the frontend (phone) code for the e-mission server
https://lafabriquedesmobilites.fr/tracemob/
BSD 3-Clause "New" or "Revised" License
9 stars 1 forks source link

Distinguish between Air and High Speed Train #13

Open PatGendre opened 3 years ago

PatGendre commented 3 years ago

Pour l'instant il y a un mode TGV/avion ce qui n'est pas une bonne chose si on veut calculer des bilans carbone ! Voir si on peut les différencier par la vitesse et/ou par les gares/aéroports de départ/arrivée

PatGendre commented 3 years ago

Le code est ici https://github.com/e-mission/e-mission-server/blob/2177582fdea9a52faa8c1a238e62b111dbbcedba/emission/analysis/intake/cleaning/clean_and_resample.py#L508 Une 1ère idée très simple serait de remonter le seuil de vitesse pour l'avion de 150 à 400km/h, et de regrouper le TGV avec le train et changer le mode AIR_or_HSR en AIRPLANE

PatGendre commented 3 years ago

dans is_air_section

PatGendre commented 3 years ago

la liste des modes détectés est ici : https://github.com/e-mission/e-mission-server/blob/714010ad0990209117202234618360f7333dbac3/emission/core/wrapper/modeprediction.py#L17 Le mode s'appelle AIR_OR_HSR mais on veut qu'il corresponde en fait à Avion. Sans changer le nom de la constante pour l'instant...

Il faut alors modifier ce fichier : https://github.com/fabmob/tracemob-phone/blob/9e29c6018e839fccfb348037a5204316ccc1c209/www/i18n/en.json#L364 et ce fichier https://github.com/fabmob/tracemob-phone-translate/blob/master/fr/i18n/fr.json

PatGendre commented 3 years ago

Idéalement il faudrait tester sur un serveur local avant de committer mais si on est pressé ce n'est pas risqué !

iciatech-yprime commented 3 years ago

J'ai fait des tests sur le serveur, tout marche, j'ai mergé avec la branch tgv. Je revérifierai demain arpès le pipeline de cette nuit si tout s'est bien passé

PatGendre commented 3 years ago

@iciatech-yprime super! on verra demain si on peut clore l'issue