Mieux gérer les unités Publicodes

Clemog commented 2 years ago

On devrait travailler le système d'unités du modèle:

Pour travailler l'homogénéité des calculs pour détecter les erreurs
Pour les conversions (genre minutes en heures, wh en kwh)
Egalement les "s" (genre heure/heures)

Il semble que des travaux et discussions sont déjà en cours coté Publicodes (et même certaines solutions sont déjà en ligne), tu confirmes @laem ?

Tasks

[x] Repasser sur les warnings et établir une liste exhaustive de ce qui manque côté Publicodes pour les fixer. (1 j)
[x] Décoréller les unités utilisées pour le calcul de celles utilisées dans l'UI. (1/2 j)

Les cas non gérés

Pb avec les unités des règles non applicables (exemple climatiseur)
Les unités "puissance" : x m * y m devrait donner xy m2 -> puissance dans publicodes ?
Conversion ds kgCO2e / an en kgCO2e car le résultat est donné annuellement finalement. Idem / personne pour l'amortissement foyer
Lié au point précédent avec l'exemple de la question dépenses : on souhaite afficher à côté de l'input l'unité €/an. La valeur entrée ici est également amortie sur 10 ans, on a donc une unité en kgCCO2e/an.an au lieu de kgCO2e
On ne gère pas bien les pluriels : heures ans -> choisir un unité singulière ? Dans ce cas comment gérer l'affichage de l'unité dans le simulateur ?
Une formule pourtant homogène du type :
```
formule: consommation journalière * jours par an * empreinte cigarette
unité: kgCO2e
```
Renvoie : [ Avertissement ] ➡️ Dans la règle "divers . tabac" ⚠️ Erreur lors de la conversion d'unité explicite ℹ️
[ EngineError ] ✖️ Impossible de convertir l'unité 'cigarettes / jour' en 'kgCO2e'

Avancement 01/08

Les erreurs sont gérée pour l'évaluation initiale de bilan sauf pour:
- Les erreurs d'unités dans les modèles importés
- La problématique du voyageur pour l'amortissement de la voiture
On traite les variables utiles pour l'amortissement (an, habitant) sans unité (sauf pour voyageur -> besoin ui).
On traite seulement le cas des valeurs par défaut à l'initialisation
Pb de conversion kW.h en kWh, idem mn/h en noeud.. On devrait avoir une table d'unité équivalente ?
De nombreuses erreurs liées aux voyageurs : amortissement ..

jeannelf commented 1 year ago

@Clemog en début de recette je vois un Bug, qui ne se produit pas en prod (peut etre que c'est normal tu me dis)

Comportement constaté sur https://deploy-preview-1275--nosgestesclimat.netlify.app/?PR=2014 : lorsque j'utilise une étiquette "pré remplie", le chiffre ne se met pas à jour dans la case https://www.loom.com/share/a0ac13d3afb141f79037644dd46207bc

Comportement en prod (et souhaité) : lorsque je clique sur une étiquette, le chiffre dans la case se modifie https://www.loom.com/share/b6376ce0c394489f9b52f19d7e4503fd

jeannelf commented 1 year ago

J'ajotue un micro détail : sur la question de l'appart, peut on écrire m² plutôt que m2 ? enfin je dis ca mais faut peut etre vérifier que ca se lit bien sur tous les formats ? cest du utf8 j'imagine ?

Clemog commented 1 year ago

en début de recette je vois un Bug, qui ne se produit pas en prod (peut etre que c'est normal tu me dis)

C'est fix ;)

Clemog commented 1 year ago

J'ajotue un micro détail : sur la question de l'appart, peut on écrire m² plutôt que m2 ? enfin je dis ca mais faut peut etre vérifier que ca se lit bien sur tous les formats ? cest du utf8 j'imagine ?

Intéressant, on a eu la réflexion sur l'unité CO2e / CO₂e, pas évident à trancher

C'est une contrainte d'écriture mais on peut effectivement se dire que c'est logique d'afficher l'unité comme elle doit l'être

Je fais la modification pour les m2

EmileRolley commented 1 year ago

Intéressant, on a eu la réflexion sur l'unité CO2e / CO₂e, pas évident à trancher

Selon moi :

CO2e est utilisé dans le code car ça permet d'unifier le format et de faciliter la rédaction et le parsing.
CO₂e peut être utilisé dans l'interface si on souhaite avoir quelque chose de plus joli ?

Même chose pour m2 et m².

incubateur-ademe / nosgestesclimat