[NE PAS FERMER]🇨🇭 Version Genevoise #1872

GRID-JPR · 2023-05-09T15:10:58Z

Bonjour,
Voici la première modification du profile Suisse/Geneva.

Closes #1941

En attendant le versionnage, suivi des ajouts en production ici : #2085

github-actions · 2023-05-11T10:28:55Z

Report for the pull request #1872

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

🔧 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 BE-en CA-en CH-en DE-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	✔️	Ø
Documentation compilation to JSON	✔️	Ø

🌐 Translation status

Rules

Language	Nb. missing translations	Status
en	Missing 15 rules ⬇️ logement . raccordement réseau de chaleur > titre logement . chauffage . fioul kWh au litre > note logement . chauffage . fioul kWh au litre > titre services marchands > abréviation services publics > abréviation transport . vacances . caravane > note transport . vacances . caravane . usage réel > titre transport . vacances . caravane . construction amortie > titre transport . vacances . caravane . construction > note transport . vacances . caravane . construction > titre transport . vacances . camping car . usage réel > titre transport . vacances . camping car . construction amortie > titre transport . vacances . camping car . construction > note transport . vacances . camping car . construction > titre transport . vacances . camping car . usage > titre	❌
en	Missing 10 rules (not up-to-date) ⬇️ fioul kWh au litre transport . vacances . camping car . empreinte transport . vacances . camping car . empreinte . construction transport . vacances . camping car . empreinte . construction amortie transport . vacances . camping car . empreinte . usage transport . vacances . camping car . empreinte . usage réel transport . vacances . caravane . empreinte transport . vacances . caravane . empreinte . construction transport . vacances . caravane . empreinte . construction amortie transport . vacances . caravane . empreinte . usage réel	❌

Personas

Language	Nb. missing translations	Status
en	Ø	✔️

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)	Δ (%)
	13,355	13,355	-0%

Test model optimisation

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

Clemog

Première revue rapide ! Merci pour ces premières modifications !

data/i18n/models/CH-fr.yaml

Clemog · 2023-07-04T13:15:36Z

Hello @GRID-JPR :)

Merci pour ces nouvelles modifications. J'ai fait quelques retours plus haut, je remonte 3 petits points plus généraux:

Je vois que dans certains cas, seul un des attributs de la règle est réécrit par rapport au modèle de base : ce n'est pas nécessaire de laisser tous les attributs de la règle de base dans le fichier de configuration mais seulement ceux que vous souhaitez modifier (exemple: si question n'est pas modifié, ce n'est pas nécessaire de le remettre).
Dans le cas ou une formule est un calcul, c'est possible de remplacer directement la formule par une constante (je pense au train ou à la surface du logement).
Je vois peu de sources de données, il me semble important de les proposer pour plus de transparence pour l'utilisateur ou d'autres ré utilisateurs du modèle

Clemog · 2023-07-04T13:15:52Z

Et encore une fois, je suis désolé pour le délai de réponse !

GRID-JPR · 2023-09-13T14:45:51Z

data/i18n/models/CH-fr.yaml

Clément,
Il manque encore la suppression de certain rubrique dans ce commit. Comme par exemple la supression de l'estimation par le prix pour logement . chauffage . réseau de chaleur . estimation via le prix.
Peux-tu me guider pour effectuer cette modification stp.

Hello ! Je ne suis pas sur que ce soit facilement implémentable dans l'immédiat car il est nécessaire de supprimer une règle, ce que l'on ne permet pas pour le moment ..

Hello,
ok, je regarde avec Justine si l'on a pas moyen d'utiliser tout de même cette option pour la Suisse. merci

N'hésite pas à ajouter qques notes relativement à mes commentaires plus haut, je refais une passe sur les derniers ajouts la semaine prochaine :)

Bumps [ramda](https://github.com/ramda/ramda) from 0.28.0 to 0.29.1. - [Release notes](https://github.com/ramda/ramda/releases) - [Changelog](https://github.com/ramda/ramda/blob/master/CHANGELOG.md) - [Commits](ramda/ramda@v0.28.0...v0.29.1) --- updated-dependencies: - dependency-name: ramda dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>

…abot/npm_and_yarn/preprod/ramda-0.29.1 chore(deps-dev): bump ramda from 0.28.0 to 0.29.1

…or-contenu-ecrit refactor(docs): move contenu-ecrit/ to nosgestesclimat-site-nextjs

…esclimat into preprod

Clemog reviewed May 11, 2023

View reviewed changes

data/i18n/models/CH-fr.yaml Show resolved Hide resolved

data/i18n/models/CH-fr.yaml Outdated Show resolved Hide resolved

data/i18n/models/CH-fr.yaml Outdated Show resolved Hide resolved

data/i18n/models/CH-fr.yaml Outdated Show resolved Hide resolved

GRID-JPR force-pushed the geneva branch from 0c723eb to 9643374 Compare June 12, 2023 14:47