search

ziyi105 / pe

0 stars 0 forks source link

calories round up to 0 for very small calories #9

Open ziyi105 opened 9 months ago

ziyi105 commented 9 months ago

image.png

Perhaps you could add a lower limit for calories and output a specific error message for very low calories?

nus-pe-bot commented 9 months ago

Team's Response

Thank you for the report. But the app has the value inside its memory, but shows as rounded value to the user for readability. This is exactly what we intended.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: How would the user know the value is indeed recorded in the memory? Without knowing this information, the user might think that this app is unable to accept value that is too small, hence will never input small value again. Furthermore, the team mentioned "this is exactly what we intended", which implies that they intend to display 0kcal to the user. As a developer, I think this is not very user friendly to show 0 when there is actually a value there.