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

Ch model adaptations #2454

Merged
merged 7 commits into from
Oct 17, 2024
Merged

Conversation

trepmag
Copy link
Contributor

@trepmag trepmag commented Sep 26, 2024

Here are some adaptions requested by the Canton and Ville de Genève and was adapted from model version 2.5.6...

Note that we noticed that nosgestesclimat-site-nextjs (v2.0.1) is failing to build (yarn build) when using that model adaptation with the following output:

$ yarn build
yarn run v1.22.22
$ next build
   Next.js 14.2.7
  - Experiments (use with caution):
    · turbo
    · webpackBuildWorker
    · instrumentationHook

   Creating an optimized production build ...
  Compiled successfully
   Linting and checking validity of types  .Failed to compile.

./src/app/(simulation)/simulateur/[root]/_components/simulateur/charts/inlineChart/SubcategoriesChart.tsx:43:37
Type error: Parameter 'subcategory' implicitly has an 'any' type.

  41 |           currentCategory
  42 |         )}`}>
> 43 |         {filteredSubcategories.map((subcategory, index: number) => (
     |                                     ^
  44 |           <Subcategory
  45 |             key={subcategory}
  46 |             position={
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

@bjlaa
Copy link
Contributor

bjlaa commented Sep 26, 2024

That's weird, I don't get the same error ; I would advice you to delete the .next folder along with the next-env.d.ts and the node_modules ; re-run yarn then yarn build.

@trepmag
Copy link
Contributor Author

trepmag commented Sep 26, 2024

That's weird, I don't get the same error ; I would advice you to delete the .next folder along with the next-env.d.ts and the node_modules ; re-run yarn then yarn build.

I did but by adding yarn link @incubateur-ademe/nosgestesclimat after yarn so that this model adaptation is being included during the build; but unfortunately this didn't resole the issue...

@bjlaa
Copy link
Contributor

bjlaa commented Sep 26, 2024

Hmm I guess the only option is to dive into the site repo and correct the type error, from here this looks like something that could be solved by adding some any in the file but I can't be sure.

@trepmag
Copy link
Contributor Author

trepmag commented Sep 26, 2024

Ok, thank you; will do that...

@bjlaa bjlaa self-requested a review September 30, 2024 13:30
@bjlaa
Copy link
Contributor

bjlaa commented Sep 30, 2024

Le modèle semble compiler correctement. @JuliePouliquen est-ce que tu voudrais jeter un coup d'oeil également ?

data/i18n/models/CH-fr.publicodes Outdated Show resolved Hide resolved
data/i18n/models/CH-fr.publicodes Outdated Show resolved Hide resolved
@@ -371,6 +453,7 @@ parc français . chauffage . consommation réseau de chaleur par m2:

# @Clément On a besoin des parts de chaque type d'énergie dans le parc genevois. Il semble y avoir un pb avec ceux relevés dans le fichier.
# On distingue parts des énergies dans le parc genevois et part dédiée au chauffage pour chaque type d'énergie (cf plus bas).
# @JPhilippe Ici ce sont bien les parts de chaque combustibles pour le chauffage. Je n'ai pas compris comment nous devons ajuster ces chiffres pour que le modèle fonctionne ?
Copy link
Contributor

Choose a reason for hiding this comment

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

Dans le modèle de base, ces données (qui servent à calculer l'empreinte moyenne du chauffage d'un m² de logement, tous systèmes de chauffage confondus, au prorata de leur usage en France, pour les personnes qui répondent "je sais pas" à la question sur leur mode de chauffage) sont calculées à partir des consommations nationales de gaz, fioul, bois, réseau de chaleur, rapportées aux surfaces chauffées par ces mêmes moyens. Je dirais qu'idéalement, il faudrait remplacer toutes ces valeurs pour correspondre au parc de logements suisses, et en même temps... on peut peut-être partir du principe qu'en moyenne le chauffage des logements suisse est similaire à celui des logements français.
Ceci étant dit, je ne sais pas pourquoi il faut changer ces valeurs pour que le modèle fonctionne...

Copy link
Contributor

Choose a reason for hiding this comment

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

Pour calculer l'empreinte par défaut d'un logement, nous avons fait le calcul suivant :

Somme pour chaque combustible (part du chauffage par combustible (%) * consommation par défaut (kWh/m2))

Donc le mode de calcul est proche mais ne permet pas directement de remplacer les valeurs, on a cru comprendre qu'il nous fallait directement remplacer la valeur par défaut. Voici notre tableau de calcul :

 

 

  Ajusté GE  
  Unité FE
Logement    
logement . chauffage . empreinte par défaut kgCO2e 2'373
parc français . chauffage . consommation gaz moyenne relative kWh 6'080
parc français . chauffage . consommation fioul moyenne relative kWh 2'840
parc français . chauffage . consommation bois moyenne relative kWh 230
parc français . chauffage . consommation réseau de chaleur moyenne relative kWh 520
parc français . chauffage . consommation chaleur de l'environnement moyenne relative kWh 330
parc français . chauffage . consommation électricité totale relative kWh 165
parc français . chauffage . consommation GPL moyenne relative kWh -
logement . chauffage . intensité carbone chaleur de l'environnement kgCO2e/kWh 0.040
parc ge. chauffage . consommation par défaut kWh/m2 125
logement . part chauffage . fioul % 28.40%
logement . part chauffage . gaz % 60.80%
logement . part chauffage . réseau de chaleur % 5.20%
logement . part chauffage . chaleur de l'environnement % 3.30%
logement . part chauffage . bois % 2.30%

 

On peut s'aligner sur le modèle FR mais on n'avait pas bien compris quelles valeurs il faut fournir afin que le modèle tourne correctement. C'est intéressant je pense de garder la spécificité GE pour le modèle de chauffage.

data/i18n/models/CH-fr.publicodes Outdated Show resolved Hide resolved
data/i18n/models/CH-fr.publicodes Outdated Show resolved Hide resolved
data/i18n/models/CH-fr.publicodes Outdated Show resolved Hide resolved
description: |
On peut être un grand adepte de la marche, du vélo et des transports en communs et avoir parfois besoin d’une voiture ; pour les loisirs, les déplacements longs ou pour aller faire de grosses courses.

L’autopartage est une solution, en particulier si vous ne voyez pas l’intérêt d’être propriétaire d’un véhicule qui passe le plus clair de son temps au parking. Dans le cadre de l’autopartage, plusieurs personnes utilisent le même véhicule, il est donc partagé. Ce sont des coûts en moins car eux aussi sont partagés (assurance, charges d’entretien, place de parking).
Copy link
Contributor

Choose a reason for hiding this comment

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

Peut-être un mot sur les offres d'autopartage de Genève, s'il y en a ? C'est l'intérêt d'être très géolocalisé.

Copy link
Contributor

Choose a reason for hiding this comment

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

ok on rajoute: À Genève il est par exemple possible de souscrire un abonnement mobility.

data/i18n/models/CH-fr.publicodes Outdated Show resolved Hide resolved
@bjlaa bjlaa changed the base branch from master to preprod October 11, 2024 09:54
@bjlaa bjlaa merged commit f0ab5f2 into incubateur-ademe:preprod Oct 17, 2024
6 of 7 checks passed
bjlaa added a commit that referenced this pull request Oct 17, 2024
* Modifs linge + cuisine

* Suppression division par foyer

* Voiture propriétaire par défaut

* Hausse lessives par habitant

* Fix par défaut voiture

* par défaut voiture modif

* Plus d'eau jardin

* Hausse cuisine + vaisselle

* Encore plus de lessives

* Ajout lien ressource ADEME eau domestique

* Traductions

* Ajout de notes

* Français devient humain

* Traductions

* Fix trads

* ✨ Add types for Categories and Subcategories (#2458)

* ✨ Add types for Categories and Subcategories

* ♻️ Remove unused var

* Ch model adaptations (#2454)

* Modifications CH-fr model June 2024

* Add notes and small modification CH-fr July 2024

* Modification August on Actions/agir

* Fix CH-fr model to renamed base rules keys + translate CH-es and CH-en using DeepL

* Minor Fix requested by NGC.fr team

---------

Co-authored-by: jrichard <[email protected]>
Co-authored-by: Benjamin Arias <[email protected]>

* 🔖 Release 3.3.0

* 🗑️ Remove unused translations

---------

Co-authored-by: JuliePouliquen <[email protected]>
Co-authored-by: Cedric Gampert <[email protected]>
Co-authored-by: jrichard <[email protected]>
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