incubateur-ademe / bilansclimatsimplifies

MIT License
1 stars 1 forks source link

Ecran récapitulatif du bilan (fanny) #32

Closed Clem-mrtn closed 2 years ago

Clem-mrtn commented 2 years ago

image image

hfroot commented 2 years ago

@Clem-mrtn tous les FEs sont données comme "kgCO2e/<unité>" - est-ce que cette rêquete c'est pour une conversion en finale pour kg -> tonne ? Si oui, est-ce que tu peux confirmer que c'est 1kg = 0,001 tonne ?

florianpanchout commented 2 years ago

@hfroot Je m'occupe de cette issue en front. Ça me semble plus simple / pertinent de continuer nos échanges en kgCO2e, et que je m'occupe de l'affichage en tCO2Ee non ?

hfroot commented 2 years ago

@florianpanchout J'ai besoin quand même de calculer en tCO2e pour les exports. Je peux rendre ces calculs dispo dans le API total_t, poste_x_t si tu veux les utiliser (ou simplement changer les total et poste_x directement).

florianpanchout commented 2 years ago

Nope, pas besoin. Je fais déjà le calcul avec les kg (je doute que l'on ai une différence entre ton calcul et le mien ?)

hfroot commented 2 years ago

Petit retour - je pense ça peut être utile d'avoir un précision sur le champ pour les totals manuels que c'est max 3 chiffres significatifs. Maintenant si on mis plus, tout marche mais les derniers chiffres sont perdus.

Clem-mrtn commented 2 years ago

Pour les personnes qui ont déjà fait leur bilan et qui renseignent à la main les posts 1 & 2 ? Mettre une limite de 3 chiffre avant ou après la virgule ?

Clem-mrtn commented 2 years ago

Attention, je ne sais pas si on aura beaucoup de personnes dans ce cas, mais pour les petits résultats d'émissions, on se retrouve avec 0 tonne de CO2 affiché sur l'écran Bilan des Post 1 & 2 => cf. screen ci-dessous, j'étais à 145 kg de CO2 pour le post 2 et dans le récap là je suis à 0

image

=> Infos entreprise (tjrs en petit comme ça) => Post 1 & Post 2 => Publier ce bilan qu'en pensez vous ?

hfroot commented 2 years ago

Pour les personnes qui ont déjà fait leur bilan et qui renseignent à la main les posts 1 & 2 ? Mettre une limite de 3 chiffre avant ou après la virgule ?

Après le virgule - par ex "10,123" et jamais "10,1234". La validation côté back est déjà là - il coupe "10,1234" pour être "10,123".

À propos les petits chiffres - est-ce qu'on peut calculer/demander si c'est un cas probable ?

florianpanchout commented 2 years ago

J'ai enlevé l'arrondi pour les chiffres inférieurs à 1 tonne (on garde l'arrondi du back)

florianpanchout commented 2 years ago

Tout est traité normalement

florianpanchout commented 2 years ago

Ah non pas cette partie :

Et + pour le coup j'aurais mis les informations de l'entreprise au dessus des résulats /poste => Infos entreprise (tjrs en petit comme ça) => Post 1 & Post 2 => Publier ce bilan qu'en pensez vous ?

À noter que c'était comme ça avant et que je le change suite à cette issue (https://github.com/datagir/bilansclimatsimplifies/issues/12) que j'ai lu trop vite. Perso ça me semble logique d'afficher les totaux en premier. C'est ce que les gens veulent vérifier, le reste ils connaissent déjà.

Clem-mrtn commented 2 years ago

=> changer l'ordre ID Résultat Bilan

florianpanchout commented 2 years ago

Done