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

Ajout des bornes (NGC-330) #2224

Merged
merged 20 commits into from
Feb 13, 2024
Merged

Ajout des bornes (NGC-330) #2224

merged 20 commits into from
Feb 13, 2024

Conversation

Clemog
Copy link
Contributor

@Clemog Clemog commented Dec 14, 2023

  • Ajout de tests unitaires (partiels)
  • Ajout de planchers à tous les inputs pertinents
  • Pas d'ajout de plafond pour le moment (sauf pour un %)
  • Gérer côté site les 2 notifs voyageurs & habitants

Copy link

Borner les inputs

Copy link

github-actions bot commented Dec 14, 2023

Report for the pull request #2224

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


🔧 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
  • NC-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
  • NC-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 ✔️ Ø
Migration instructions compilation to JSON ✔️ Ø

🌐 Translation status

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

Rules

Language Nb. missing translations Status
en Ø ✔️

Personas

Language Nb. missing translations Status
en Ø ✔️

Test translation (fr -> en)

Règle fr en

👫 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) Δ (%)

Test the model optimisation

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

Copy link
Contributor

@JuliePouliquen JuliePouliquen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ça me paraît top, je me demande si on n'ajoute pas des plafonds à pas mal de questions, même si parfois ils seraient assez arbitraires. Mais juste pour les zozos qui voudraient tester.
Genre plafond chiens chats 100 (ça me paraît pas mal), thé café... 35, repas 21... T'en dis quoi ?

On pourrait aller chercher ces plafonds dans Matomo peut-être : prendre le max et ajouter 20% ?

@Clemog
Copy link
Contributor Author

Clemog commented Dec 18, 2023

Ça me paraît top, je me demande si on n'ajoute pas des plafonds à pas mal de questions, même si parfois ils seraient assez arbitraires. Mais juste pour les zozos qui voudraient tester. Genre plafond chiens chats 100 (ça me paraît pas mal), thé café... 35, repas 21... T'en dis quoi ?

A voir dans un second temps peut etre mais effectivement ce serait arbitraire..

On pourrait aller chercher ces plafonds dans Matomo peut-être : prendre le max et ajouter 20% ?

On peut éventuellement regarder du côté des données de l'enquête mais via matomo, nous n'avons pas de données par question

@JuliePouliquen
Copy link
Contributor

OK alors ça me va !

@Clemog
Copy link
Contributor Author

Clemog commented Dec 18, 2023

J'ajoute ne sug ton idée !

@Clemog
Copy link
Contributor Author

Clemog commented Dec 18, 2023

L'intégration de ces bornes n'est pas dingue niveau site:

  • Pas d'indication si valeur interdite
  • La conversion vers la valeur "minimale" se fait de manière étrange ..

Un avis @florianpanchout ?

@Clemog
Copy link
Contributor Author

Clemog commented Dec 18, 2023

(ce sera pour la prochaine release)

@JuliePouliquen
Copy link
Contributor

Il faudrait rajouter les planchers pour l'électroménager et le mobilier, non ? (cf. ton post sur Mattermost).

@Clemog
Copy link
Contributor Author

Clemog commented Jan 8, 2024

Il faudrait rajouter les planchers pour l'électroménager et le mobilier, non ? (cf. ton post sur Mattermost).

Oups, oui surement un oubli!

@JuliePouliquen
Copy link
Contributor

Oups, oui surement un oubli!

Edit : ils sont bien intégrés. C'est juste que ce n'est pas encore actif !

Copy link
Contributor

@JuliePouliquen JuliePouliquen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nickel. Mais à lier à de l'affichage, comme tu le disais ?

tests/unit/bornes.test.mjs Show resolved Hide resolved
@Clemog
Copy link
Contributor Author

Clemog commented Jan 9, 2024

Nickel. Mais à lier à de l'affichage, comme tu le disais ?

Oui unee carte à gérer côté site

Copy link

vercel bot commented Feb 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nosgestesclimat-modele ✅ Ready (Inspect) Visit Preview Feb 13, 2024 11:00am

@Clemog Clemog merged commit 9cd0949 into preprod Feb 13, 2024
8 checks passed
@Clemog Clemog deleted the ajout-bornes branch February 13, 2024 11:01
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.

4 participants