nus-cs2113-AY2425S1 / pe-dev-response

0 stars 0 forks source link

No max mark handling #358

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

image.png

It seems like you should have exception handling extremely large numbers (like numbers of 10000) As per your user guide

image.png


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

yeekian commented 1 week ago

Team's Response

Thank you for your response, it is definitely a good idea to give a warning to the professor should the input max score be extremely large. We did mention the max score limit in the user guide as a form of guideline for the professor.

We do not want to do overzealous restrictions on the max score of a component as we would like the professor to decide what the max score should be. This provides flexibility for the way the professor would like to use the app.

Additionally, the max score was recorded correctly as 1.0E256.

Lastly, as stated in the PE documentation, we can classify this input as extreme user behaviours, as such large input values would only happen in rare cases, hence it should not be considered a bug.

Note: The issue can be reproduced.


image.png

image.png

Duplicate status (if any):

--