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

Nouvelles actions alimentation - NGC-306 / NGC-482 / NGC-424 #2223

Merged
merged 65 commits into from
Jan 18, 2024

Conversation

JuliePouliquen
Copy link
Contributor

@JuliePouliquen JuliePouliquen commented Dec 14, 2023

Fix #2023

  • Faire en sorte que des personas soient concernés par ces actions.
  • Vérifier auprès des personas l'apparition et le calcul des actions.
  • Traductions

En bref :

Repas

Nouvelles actions:

  • Privilégier les poissons comme le thon, le saumon, la sardine ou le maquereau (note Clément: me semble trop long)
  • Viande max une fois par jour
  • Viande max 2 fois par semaine
  • Poisson max 2 fois par semaine
  • Ajout d'un message d'attention pour le régime végétalien, non adapté à tous.

Boissons

Nouvelles actions:

  • Se limiter à deux cafés par jour
  • Se limiter à un café par jour
  • Remplacer un café quotidien par un thé
  • Remplacer le café par du thé
  • Remplacer le café par de la chicorée
  • Se limiter à un chocolat chaud par jour
  • Consommer des sirops plutôt que des sodas ou jus
  • Ne pas boire plus d'un litre de jus et sodas par semaine
  • Ne pas boire plus d'un litre d'alcool par semaine

A réfléchir:

  • Consommer du lait végétal (compliqué car produits laitiers non variabilisés, possible de le calculer pour le cholocat chaud uniquement..)

Modifications des personas

Des personas ont été modifiés pour que les nouvelles actions se retrouvent chez au moins un d'entre eux. Les principales modifications concernent Nolan dont j'ai augmenté de manière importante la consommation de boissons sucrées et chocolat chaud, ce qui alourdit notablement sa moyenne.

Copy link

Copy link

github-actions bot commented Dec 14, 2023

Report for the pull request #2223

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


🔧 Model compilation status

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

🌐 Translation status

Rules

Language Nb. missing translations Status
en Ø ✔️

Personas

Language Nb. missing translations Status
en Ø ✔️

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


👫 Personas changes

Test personas bilans against production

Persona Total PR (kg CO2e) Total in prod. (kg CO2e) Δ (%)
4,859.354 4,803.02 +1%
9,032.314 9,013.426 +0%
7,237.239 7,071.755 +2%
7,055.622 6,548.419 +8%

Test personas bilans against preprod

Persona Total PR (kg CO2e) Total in prod. (kg CO2e) Δ (%)
4,859.354 4,803.02 +1%
9,032.314 9,013.426 +0%
7,237.239 7,071.755 +2%
7,055.622 6,548.419 +8%

Test the model optimisation

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

@Clemog
Copy link
Contributor

Clemog commented Dec 20, 2023

Je viens de regarder le code très rapidement (pas eu le temps de tester)

  • Top pour le "nombre de semaines par an"
  • Pas du tout pour le changement d'intitulé des actions "diviser par 2" etc.. mais à discuter
  • Ce serait pas mal d'ajouter des unités désormais pour les actions aussi (on l'a jamais fait) cf feat: add check for action units #2242
  • Je crois que les recalcul que tu utilises ne sont pas utiles dans les différents cas et je trouve que ça alourdit un peu la syntaxe. La valeur de l'action pourrait être simplement la diff entre ce qui est consommé et la valeur "max" x l'empreinte, c'est plus simple et clair à mon avis

Sinon très cool de voir ces ajouts ! 👌

data/actions/alimentation.publicodes Outdated Show resolved Hide resolved
data/actions/alimentation.publicodes Outdated Show resolved Hide resolved
data/actions/alimentation.publicodes Outdated Show resolved Hide resolved
data/actions/alimentation.publicodes Outdated Show resolved Hide resolved
data/actions/alimentation.publicodes Outdated Show resolved Hide resolved
data/actions/alimentation.publicodes Outdated Show resolved Hide resolved
@Clemog Clemog self-requested a review December 20, 2023 16:44
@Clemog
Copy link
Contributor

Clemog commented Jan 16, 2024

J'ai fait un dernier tour, je te laisse vérifier si ça te convient ?

Tu avais vérifier en ligne ce que ça donnait ?

@JuliePouliquen
Copy link
Contributor Author

JuliePouliquen commented Jan 18, 2024

J'ai fait un dernier tour, je te laisse vérifier si ça te convient ?

Tu avais vérifier en ligne ce que ça donnait ?

Oui j'avais vérifié, notamment avec les personas qui étaient bien affectés par la PR :).

Du coup... on est bons ? 😃

@Clemog
Copy link
Contributor

Clemog commented Jan 18, 2024

Il manque des trads mais sinon oui :)

@Clemog
Copy link
Contributor

Clemog commented Jan 18, 2024

Mais en l'état et en testant j'ai l'impression que l'écran action est inutilisable ..

image

@Clemog
Copy link
Contributor

Clemog commented Jan 18, 2024

Et les questions supplémentaires ne s'affichent jamais

@JuliePouliquen
Copy link
Contributor Author

Il manque des trads mais sinon oui :)

Aaaaah les trads, toujours les trads !

C'est à cause de nous les bugs de l'écran action tu crois ?

@Clemog
Copy link
Contributor

Clemog commented Jan 18, 2024

Nope .. pas à cause de la PR j'ai fait des suggestions mais ça fait longtemps je crois ..

@JuliePouliquen
Copy link
Contributor Author

Nope .. pas à cause de la PR j'ai fait des suggestions mais ça fait longtemps je crois ..

OK. Je me suis occupée des trads, normalement c'est bon.

@JuliePouliquen JuliePouliquen merged commit 187c8de into preprod Jan 18, 2024
7 checks passed
@JuliePouliquen JuliePouliquen deleted the nouvelles-actions-alimentation branch January 18, 2024 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants