Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Distinguish between Air and High Speed Train #13

Open
PatGendre opened this issue Apr 1, 2021 · 6 comments
Open

Distinguish between Air and High Speed Train #13

PatGendre opened this issue Apr 1, 2021 · 6 comments
Labels
enhancement New feature or request testing testing before delivery

Comments

@PatGendre
Copy link

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 PatGendre added the enhancement New feature or request label Apr 1, 2021
@PatGendre
Copy link
Author

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
Copy link
Author

dans is_air_section

  • remplacer HUNDRED_KMPH par THREE_HUNDRED_KMPH
  • et ONE_FIFTY_KMPH par FOUR_HUNDRED_KMPH
  • quantile 0.9 par 0.7

@PatGendre
Copy link
Author

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 :

"AIR_OR_HSR": "Air or HSR",

et ce fichier https://github.com/fabmob/tracemob-phone-translate/blob/master/fr/i18n/fr.json

@PatGendre
Copy link
Author

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

@iciatech-yprime iciatech-yprime added the testing testing before delivery label Jun 14, 2021
@iciatech-yprime
Copy link

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
Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request testing testing before delivery
Projects
None yet
Development

No branches or pull requests

2 participants