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 résidences secondaires (NGC-221) #2190

Merged
merged 55 commits into from
Dec 14, 2023

Conversation

JuliePouliquen
Copy link
Contributor

@JuliePouliquen JuliePouliquen commented Nov 16, 2023

Fix #411

Todo :

  • Ajouter le chauffage
  • Ajouter de la résidence secondaire aux personas
  • Ajouter la question "spéciale" pour les nuitées avec deux champs semaines / week-ends
  • Traduction
  • Bug mosaique choix logement vacances -> fixé avec un hack ici : TEMP fix: mosaic child need to includes keys nosgestesclimat-site-nextjs#249, sera fixé après implé nouvele spec des mosaique
  • Checker l'ordre des questions
  • Utiliser le mécanisme avec pour alléger le code
  • Vérifier l'homogénéité des unités
  • Ajouter une empreinte par défaut pour la résidence secondaire

Copy link

@JuliePouliquen JuliePouliquen changed the title Avancées résidence secondaire (NGC-221 Avancées résidence secondaire (NGC-221) Nov 16, 2023
Copy link

github-actions bot commented Nov 17, 2023

Report for the pull request #2190

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


🔧 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) Δ (%)
9,267.809 9,152.612 +1%
2,722.784 2,758.584 -1%
4,756.637 4,776.47 -0%
9,000.977 9,232.243 -3%
11,844.769 11,666.823 +2%
7,941.454 7,936.474 +0%
7,092.21 7,128.011 -1%
6,492.325 6,366.581 +2%
9,254.007 9,261.167 -0%
13,263.534 13,270.694 -0%

Test personas bilans against preprod

Persona Total PR (kg CO2e) Total in prod. (kg CO2e) Δ (%)
9,267.809 9,214.282 +1%
11,844.769 11,666.823 +2%
6,492.325 6,359.421 +2%

Test the model optimisation

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

@JuliePouliquen
Copy link
Contributor Author

Youhou c'est fonctionnel ! Bon il manque des descriptions pour expliquer notamment la notion de "facteurs saison" mais franchement je suis tout de même déjà bien contente.

@JuliePouliquen
Copy link
Contributor Author

Bon alors, ça marche... si ce n'est que quand on coche "résidence secondaire", temporairement, le compteur affiche 0 tonnes de CO2. Pourquoi ? Bonne question.
Aussi, je ne suis pas fan de l'ordre des questions. J'aurais dit dans l'ordre :

  1. Localisation
  2. Surface
  3. Saison
  4. Nombre de nuitées

@JuliePouliquen
Copy link
Contributor Author

JuliePouliquen commented Nov 20, 2023

Pour les calculs de valeurs par défaut :

image

Taille :

  • Source En moyenne plus petit que les résidences principales (27% font moins de 40 m³ contre 10% des résidences principales), particulièrement le cas à la montagne (50 % au-dessus de 1500 m)
  • Près de 1,9 million (environ 57 %) des résidences secondaires sont des maisons et 1,5 million (43 %) des appartements (source)

@Clemog
Copy link
Contributor

Clemog commented Nov 22, 2023

  • Ajouter la question "spéciale" pour les nuitées avec deux champs semaines / week-ends

C'est à dire ? Pourquoi pas une mosaïque ?

data/logement/vacances.publicodes Outdated Show resolved Hide resolved
data/logement/vacances.publicodes Outdated Show resolved Hide resolved
data/logement/vacances.publicodes Outdated Show resolved Hide resolved
data/logement/vacances.publicodes Outdated Show resolved Hide resolved
data/logement/vacances.publicodes Outdated Show resolved Hide resolved
data/logement/vacances.publicodes Outdated Show resolved Hide resolved
@JuliePouliquen
Copy link
Contributor Author

Corrections effectuées !

@Benjamin-Boisserie-ABC
Copy link
Collaborator

Merci @JuliePouliquen pour le retour

On peut, on avait juste choisi collectivement de limiter fortement le nombre de questions, et de voir a posteriori si les gens réagissaient à ces limites (genre "mais moi j'ai mis une PAC et je vois que j'ai du fioul dans mon mix !!").

Ok ça marche.

Oups je vais vérifier, mais il me semble qu'on était sur des sommes mensuelles et non quotidiennes pour un mois donné.

Du coup je vois toujours les mêmes erreurs sur les facteurs de saison. Est-ce que c'est moi qui me trompe ? Ou tu n'as pas encore corrigé ?

@JuliePouliquen
Copy link
Contributor Author

Du coup je vois toujours les mêmes erreurs sur les facteurs de saison. Est-ce que c'est moi qui me trompe ? Ou tu n'as pas encore corrigé ?

Pfiou j'ai fait chauffer mon cerveau et enfin compris l'erreur ! Je pense que là c'est corrigé.

@JuliePouliquen
Copy link
Contributor Author

JuliePouliquen commented Dec 12, 2023

@Clemog curieuse que tu checkes ce que j'ai fait pour rajouter l'empreinte par défaut dans la somme par défaut ! J'ai une erreur d'unité que je n'arrive pas à résoudre.

Edit : j'ai résolu un conflit mais je me perds pour résoudre le deuxième.

@Clemog Clemog force-pushed the ajout-maison-secondaire branch from d3770c9 to b4812fd Compare December 14, 2023 17:17
@Clemog
Copy link
Contributor

Clemog commented Dec 14, 2023

Pas un impact énorme sur les personas au final mais intéressant !

@JuliePouliquen
Copy link
Contributor Author

JuliePouliquen commented Dec 14, 2023

Wow @Clemog merci pour toutes ces corrections je commençais à être un peu perdue. J'ai été lire et je crois que j'ai compris une bonne partie de mes erreurs.

OK j'étais très intriguée par le poids par défaut pour Marie. 8 kg ça me paraissait improbable, alors que 13% des français disposent d'une résidence secondaire, donc ont a minima 13% * empreinte construction résidence secondaire moyenne. J'ai fouillé fouillé dans les règles, jusqu'à réaliser que... j'ai mis 0,013 au lieu de 0,13 pour le pourcentage...

@JuliePouliquen
Copy link
Contributor Author

Et sinon... ça veut dire qu'on peut merge, c'est ça ? 😍

@JuliePouliquen JuliePouliquen merged commit 7437aff into preprod Dec 14, 2023
5 checks passed
@JuliePouliquen JuliePouliquen deleted the ajout-maison-secondaire branch December 14, 2023 20:44
Copy link

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.

Résidences secondaires
4 participants