search

Andy123qq4 / pe

0 stars 0 forks source link

eat view: wrong calculation in calories summation #11

Open Andy123qq4 opened 4 months ago

Andy123qq4 commented 4 months ago

Step: input 3 foods with decimal calories

Output: even if the input can deal with decimal calories, it didnt take account the decimals into calculation of sum calories, here should be xxx.737.... image.png

nus-se-bot commented 4 months ago

Team's Response

Calories are just estimations in real situations, not expected to be overly precise. E.g all normal groceries will have an integer value of calories. If there are too many decimals it will make the UI undesirably ugly, so it is for cosmetic concern and done on purpose to type cast on integer.

Thus, we feel that this is a case of extreme user behaviour and should not be considered a bug as the lowest calorie food we can find is still an integer and calories used in our daily lives does not contain decimal place.

https://www.healthline.com/nutrition/zero-calorie-foods#5.-Broccoli

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: You can say either out of scope or you should specify that no decimal places are considered in this carolies calculation, as a user, when you are asking input to 1 decimal place, I assume you can accept decimal calories and so as the summation calculation.

You should put your response in UG or in the code, for user's understanding.

## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.High`] - [x] I disagree **Reason for disagreement:** You can change this to Low.