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

[V2] Modification des valeurs par défaut NGC-153 #2102

Merged
merged 39 commits into from
Oct 25, 2023

Conversation

Clemog
Copy link
Contributor

@Clemog Clemog commented Oct 11, 2023

Fix #2074
Fix #2025

Suite au merge de #2098, il était nécessaire de reprendre #2086 dans une nouvelle PR.

En résumé

Avec ces quelques modifications de valeurs par défaut, la valeur de départ du simulateur (persona Marie) fait un petit bond (de quelques 400 kg). Ce petit bond se décompose comme suit :

Alimentation :

  • Ajout d'une conso d'eau en bouteille par défaut : + 36 kg
  • Modifications côté boissons :
    • Café : passage de 7 à 10 cafés par semaine soit +18 kg
    • Thé : passage de 7 à 3 thés par semaine soit -1,5 kg
    • Alcool : passage de 1 à 1,5 litre / semaine soit +27 kg
    • Boissons sucrées : passage de 2 à 1 litre par semaine soit -25 kg

Divers :

  • Ajout d'une part d'animaux domestiques par défaut soit 136 kg à répartir sur les habitants du foyer contre auparavant 1 chat

Transport

  • Deux-roues : 2,7 millions de deux-roues immatriculés en France soit l'ajout par personne par défaut de 23 kg.
  • Ferry : nouvelle valeur moyenne, on passe de 80 à 12 kgCO2e par défaut

TO DO

  • Vérifier les traductions
  • Vérifier les personas
    Note: le persona Sylviane ressort car elle possède désormais un gros chien :) + petite modif du script de test ping @EmileRolley
  • Erreurs d'unités
  • J Maj gdoc Alimentation
  • J La prise en compte de la moyenne pour le camping car / caravane ne fonctionne pas de la manière dont c'est géré dans Modification des valeurs par défaut #2086 car c'est une mosaïque : besoin d'un choix utilisateur si on souhaite appliquer une petite partie à tous les utilisateurs
  • J Pour la moto par défaut: pourquoi une valeur par défaut à moto sup 250?
  • C Pour les 2 roues, actuellement, ça ne fonctionne pas car si "non usager" on a quand même la valeur par défaut.. -> Il manque le applicable si: usager comme pour l'eau en bouteille. EDIT: je me rends compte que ça ne fonctionne pas non plus pour l'eau en bouteille donc pas une bonne méthode.. Fix ici :094cc0e
  • C Cf point plus eau, revoir le calcul pour l'eau en bouteille
  • C Ajouter la moyenne pour la question booléenne de la voiture (attention on est sur une valeur par défaut thermique/essence dans tous les cas aujourd'hui) -> compliquée aujourd'hui au vu des questions de proposer un valeur moyenne sur les km en voiture, si on voulait appliquer le proncipe des deux roues et que ce soit transparent pour les utilisateurs, il faudrait rajouter une qsuestion booléenne : vous faites des km en voiture ? si oui, combien ?
  • J Manque une source pour le textile
  • J Manque une source pour les boissons
  • Pour les transports en commun, est ce qu'on est ok sur les valeurs ? typiquement le bus -> Valeurs par défaut - Transport en commun #2120
  • On traitera dans une PR séparée la question de la valeur par défaut des km en voiture -> Valeurs par défaut - Voiture #2121
  • On traitera dans une PR séparée la question de la valeur moyenne pour la mosaïque "Caravane/camping car" -> Valeurs par défaut: Transport Vacances / activation du Van ? #2123
  • On traitera dans une PR séparée le textile -> Valeurs par défaut - Textile #2122
  • On traitera dans une autre PR Valeur par défaut du ferry #2039

@github-actions
Copy link

github-actions bot commented Oct 11, 2023

Report for the pull request #2102

🚀 Test the model from the website: https://nosgestesclimat.fr?PR=2102


🔧 Model compilation status

Task Status Message
Supported regions ✔️ Ø
Successfully compiled and optimized rules:
Expand
  • FR-fr
  • BE-fr
  • CA-fr
  • CH-fr
  • DE-fr
  • GF-fr
  • GP-fr
  • IT-fr
  • LU-fr
  • MQ-fr
  • PF-fr
  • PL-fr
  • PT-fr
  • RE-fr
  • TN-fr
  • TR-fr
  • UK-fr
  • YT-fr
  • FR-en-us
  • BE-en-us
  • CA-en-us
  • CH-en-us
  • DE-en-us
  • GF-en-us
  • GP-en-us
  • IT-en-us
  • LU-en-us
  • MQ-en-us
  • PF-en-us
  • PL-en-us
  • PT-en-us
  • RE-en-us
  • TN-en-us
  • TR-en-us
  • UK-en-us
  • YT-en-us
✔️ Ø
Personas compilation to JSON for fr ✔️ Ø
Personas compilation to JSON for en-us ✔️ Ø
Documentation compilation to JSON ✔️ Ø

🌐 Translation status

Rules

Language Nb. missing translations Status
en-us Ø ✔️

Personas

Language Nb. missing translations Status
en-us Ø ✔️

You will find more information about the translation in the dedicated file.


👫 Personas changes

Test personas regression

Persona Total PR (kg CO2e) Total in prod. (kg CO2e) Δ (%)
9,321 8,932 +4%
2,875 2,836 +1%
4,863 4,796 +1%
9,351 9,250 +1%
11,747 11,718 +0%
8,068 7,574 +7%
7,213 7,203 +0%
6,345 6,494 -2%
9,336 9,268 +1%
13,355 13,163 +1%

Test model optimisation

Persona Total PR with optim. (kg CO2e) Total PR without optim. (kg CO2e) Δ (%)

@JuliePouliquen
Copy link
Contributor

JuliePouliquen commented Oct 12, 2023

  • Pour la moto par défaut: pourquoi une valeur par défaut à moto sup 250?

Le parc des deux-roues motorisés se divise en 3 catégories (source)
➢ Les cyclomoteurs (moins de 50 cm³) : 17,00 % du parc
➢ Les motocyclettes légères (50 à 125 cm³) : 26,00 % du parc
➢ Les motocyclettes lourdes (plus de 125 cm³) : 56,00 % du parc

D'où le moto sup 250 par défaut si moto oui !

@JuliePouliquen
Copy link
Contributor

JuliePouliquen commented Oct 12, 2023

* [ ]  **J** Pour les transports en commun, est ce qu'on est ok sur les valeurs ? typiquement le bus

Non, je n'ai pas trouvé grand chose là-dessus. Le pourcentage des gens qui prennent le bus régulièrement oui, après ce que donne cette régularité en temps de trajet, non. Ici, on a :

image

Une autre source indique le temps moyen passé dans les transports en commun par ville : 1h pour Paris, 40 mn pour les autres grandes villes (/ jour de semaine).

J Manque une source pour le textile

Je serais pour revenir aux précédentes valeurs en disant qu'en nombre de pièces ça fait peu mais que ça correspond à l'empreinte calculée de façon macro ?

@Clemog
Copy link
Contributor Author

Clemog commented Oct 12, 2023

Le parc des deux-roues motorisés se divise en 3 catégories (source) ➢ Les cyclomoteurs (moins de 50 cm³) : 17,00 % du parc ➢ Les motocyclettes légères (50 à 125 cm³) : 26,00 % du parc ➢ Les motocyclettes lourdes (plus de 125 cm³) : 56,00 % du parc

D'où le moto sup 250 par défaut si moto oui !

Top c'est ajouté dans la note :)

@JuliePouliquen
Copy link
Contributor

JuliePouliquen commented Oct 12, 2023

JValeur par défaut pour le ferry ?

Wow du coup j'ai re-cherché et trouvé un nombre de passagers par ferry en 2016. Déjà pas mal. 22 000 000. Donc ~ 1/3 de voyage par français. Pas de distance, par contre. Néanmoins, pour les ferries on parle bien de traversées. Les longues traversées :

  • Celles vers la Corse durent une nuit
  • Pareil pour l'Angleterre depuis la France
  • Presqu'une journée pour l'Irlande
  • Une journée pour l'Espagne depuis la Bretagne

Côté "petites traversées", plus nombreuses (nombre important de rotations, notamment en été), il s'agit d'aller aux îles qui sont juste au large :

  • Les îles bretonnes (Sein, Groix, Batz, Belle-Île, île Vierge...) : traversée moyenne une heure (en étant très large)
  • Les îles méditerranéennes : îles du Levant (1h max), Ste Marguerite et St Honorat (devant Cannes / Antibes, 30 mn).

Bref, je pense qu'en étant larges on peut se dire qu'une traversée moyenne fait 4h. Et qu'il y en a 22 000 000 par an sur l'ensemble des français.

Par contre, on exclut les croisiéristes clairement. Gros gros impact les croisiéristes, et en croissance forte. D'ailleurs ça faisait plusieurs fois que je me disais ça alors hop une nouvelle issue #2103

Autre donnée : 25 millions de voyageurs embarqués et débarqués dans les ports français, donc 4,5 millions de croisiéristes (les chiffres se recoupent).

@Clemog Clemog force-pushed the maj-valeurs-par-défaut branch from d82b56d to df4146d Compare October 12, 2023 09:47
@Clemog
Copy link
Contributor Author

Clemog commented Oct 12, 2023

Je serais pour revenir aux précédentes valeurs en disant qu'en nombre de pièces ça fait peu mais que ça correspond à l'empreinte calculée de façon macro ?

Je ne suis pas certain que ces valeurs sont fonction de l'empreinte macro

@Clemog
Copy link
Contributor Author

Clemog commented Oct 17, 2023

@JuliePouliquen je remarque que l'empreinte par défaut pour les transport . vacances ne fonctionne pas :
image

Les règles que tu appelles dans l'empreinte par défaut appartiennent à un espace nom qui est non applicable, je pense qu'on peut utiliser un recalcul ou bien reprendre le calcul à partir des règles applicables

@Clemog
Copy link
Contributor Author

Clemog commented Oct 17, 2023

J'essaie de reprendre les derniers points :)

@Clemog
Copy link
Contributor Author

Clemog commented Oct 17, 2023

Les règles que tu appelles dans l'empreinte par défaut appartiennent à un espace nom qui est non applicable, je pense qu'on peut utiliser un recalcul ou bien reprendre le calcul à partir des règles applicables

J'ai l'impression que c'est même plus compliqué que ça dans la mesure ou transport . vacances est séparé entre logement et transport .. On appelle les enfants de cette règle ..

@Clemog
Copy link
Contributor Author

Clemog commented Oct 17, 2023

@jeannelf, je me permets de remettre en haut de la pile le graphe bilan, je viens de tester en local sur l'ancienne version et on voit qu'avec les nouvelles valeurs par défaut, la voiture écrase pas mal le reste. C'est quand même un gros besoin

image

@jeannelf
Copy link

@Clemog cest deja dans le sprint :D

@jeannelf jeannelf changed the title [V2] Modification des valeurs par défaut [V2] Modification des valeurs par défaut NGC-153 Oct 18, 2023
@notion-workspace
Copy link

@jeannelf
Copy link

j'ai relu les modifs, pas de remarques particulieres ca me semble pertinent !
seule question : on est donc à l'aise de commencer avec une empreinte de 9.3T? ca ne me semble pas du tout aberrant compte tenu des moyennes mais on est passés assez vite de 8.2 à 9,3 en passant par 8.9 je veux vérifeir que tout le monde est ok @Clemog @JuliePouliquen (la question se pose tant qu'on n'a pas décidé et designé le fait de partir de zéro ou non)

@notion-workspace
Copy link

Tâche

@Clemog Clemog merged commit 673d539 into preprod Oct 25, 2023
5 checks passed
@Clemog Clemog deleted the maj-valeurs-par-défaut branch October 25, 2023 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mise à plat des valeurs par défaut Valeurs par défaut pour toutes les questions
3 participants