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

feat: #13851, #13853, #13855 update food emissions #19

Merged
merged 29 commits into from
Sep 11, 2024

Conversation

SaraKoskelainen
Copy link
Contributor

@SaraKoskelainen SaraKoskelainen commented Sep 3, 2024

Update to use new food emission calculator model

  • use new food input and result categories
  • use actual consumption amounts as inputs instead of percentages from average
  • set defaults to average consumption based on the selected diet
  • update input view
  • update mid result view
  • add info texts to users

@SaraKoskelainen
Copy link
Contributor Author

Added new colors to results DoughnutChart: brown, green, and salmon.

image

Copy link

github-actions bot commented Sep 6, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-09-11 04:07 UTC

@SaraKoskelainen
Copy link
Contributor Author

TODO, in separate PR: informing users

  • Update "Calcultation info"
  • Update mid results section texts?
  • final results view: explain why food averages are not computed even if there is enough results from other sections for average calculation
  • & update user info about coefficients changes when electricity & heating coefficients are updated too

@SaraKoskelainen SaraKoskelainen marked this pull request as ready for review September 6, 2024 12:49
@SaraKoskelainen SaraKoskelainen requested a review from a team as a code owner September 6, 2024 12:49
@SaraKoskelainen
Copy link
Contributor Author

@spaasis fronttimuutokset on mielestäni ok nyt. Api kutsujen lähettäminen ei kuitenkaan onnistu vielä ton GitHub preview kautta, koska bäkkärimuutokset on viel mergeemättä (koska esim testit on viel päivittämättä). Niin mun pitää palata tekee bäkkärimuutokset loppuun, mut ulkoasun näkee preview avulla ja tätä voi testata paikallisesti https://dev.azure.com/sykefi/Ilmastodieetti/_git/Ilmastodieettiv2/pullrequest/16576 PR:n branchia vasten tai sit venata et bäkkärimuutekset on mergetty :D

@spaasis
Copy link
Member

spaasis commented Sep 9, 2024

Oisko edelleen pointtia ruokavalion kautta pienentää näytettävien elementtien määrää? Ideaali varmaan ois pistää ne piiloon mutta käyttäjän halutessa laajennettavaksi, mutta ehkä tässä vaiheessa vois vaan piilottaa otsikkotasolla kokonaan esim. lihat vegaaneilta?

@spaasis
Copy link
Member

spaasis commented Sep 9, 2024

Voisko juomat ilmoittaa suoraan litroina, sillä oletuksella että 1l=1kg?

@spaasis
Copy link
Member

spaasis commented Sep 9, 2024

Peruna osui silmään - kuvittelisin et yläraja vois olla isompikin kuin kolme kiloa. Tiiän porukkaa jotka vetelee normiviikkonakin ainakin 4 😅

@SaraKoskelainen
Copy link
Contributor Author

Voisko juomat ilmoittaa suoraan litroina, sillä oletuksella että 1l=1kg?

Ehkäpä voisi, eihän ne ihan samat ole, eniten varmaan heittää alkoholijuomien kohdalla, mutta en tiedä onko oleellinen heitto. Toisaalta, luulisi, että käyttäjät osaavat myös arvioida määrät litroja ajatellen, vaikka kysytäänkin kiloina? Voi myös olla, että mulla on vinoutunut ajattelu tämän suhteen, koska itselle se on itsestäänselvyys ja käyttäjänä litroja kysyttäessä ehkä miettisin onkohan tiheydet otettu huomioon, mutta peruskäyttäjälle litrat ehkä kuitenkin helpottaisivat arviointia, eli vaihdan juomiin litrat.

@SaraKoskelainen
Copy link
Contributor Author

Oisko edelleen pointtia ruokavalion kautta pienentää näytettävien elementtien määrää? Ideaali varmaan ois pistää ne piiloon mutta käyttäjän halutessa laajennettavaksi, mutta ehkä tässä vaiheessa vois vaan piilottaa otsikkotasolla kokonaan esim. lihat vegaaneilta?

Eijalla oli tähän ajatuksena näyttää kaikki, jotta jos joku on lähes kasvissyöjä tai vegaani, niin saisi helpoiten lähes oikein default arvot kun osa kategorioista menee nollaan, mutta voisi kuitenkin käydä lisäämässä joillekin ruoka-aineille kulutusta. Toisaalta, "muut" kategorialle on eri päästökerroin ruokavalion mukaan, joten kannattaisin itsekin sitä, että tietyt kategoriat piilotetaan ruokavalion mukaan. Se myös ohjaisi vastaamaan oikein.

@SaraKoskelainen
Copy link
Contributor Author

Peruna osui silmään - kuvittelisin et yläraja vois olla isompikin kuin kolme kiloa. Tiiän porukkaa jotka vetelee normiviikkonakin ainakin 4 😅

Nii, ylipäänsä mietin et onko 5x riittävä max arvojen asettamiselle, koska minkä tahansa ruokaaineen suhteen on överisyöjiä :D Mutta Eija saa sanoa miten raja-arvot asetetaan.

@spaasis spaasis enabled auto-merge (squash) September 11, 2024 04:06
@spaasis spaasis merged commit 03c6e7c into main Sep 11, 2024
1 check passed
@spaasis spaasis deleted the 13851-13853-13855-Update-food-emissions branch September 11, 2024 04:07
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.

2 participants