search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Add recipes with long cook time and a lot of calories seem to fail #2517

Open nus-se-script opened 1 month ago

nus-se-script commented 1 month ago

Numbers above a certain calories or cook time seem to fail, not sure if it's as intended.

Also, the error message may be wrong as it suggest only > 1 onwards then it may work.

The user guide also did not specify the limitations of the cook time or calories.

image.png

[original: nus-cs2113-AY2324S2/pe-interim#2488] [original labels: severity.Low type.FunctionalityBug]

yuki-zmstr commented 1 month ago

Team's Response

There is some implementation of a reasonable range of calorie and cook time inputs, but I believe there is some wrap around after a certain max_int, causing the app to read it as negative. Thanks for pointing this out.

Duplicate status (if any):

--