search

dandaandaaaaaan / pe

0 stars 0 forks source link

Assessment Weightage stated in the UG says 1 to 100 inclusive, but I can create Assessment with weightage 0 #3

Open dandaandaaaaaan opened 2 years ago

dandaandaaaaaan commented 2 years ago

Screenshot 2022-04-16 at 2.27.08 PM.png

The UG states that assessment weightage should be between 1 to 100 inclusive, but I can make assessments with weightage 0.

Likewise, the error message and the UG is not sync with the spec on the allowed set weightages

Screenshot 2022-04-16 at 2.28.54 PM.png

nus-pe-script commented 2 years ago

Team's Response

Thank you for pointing out this mistake in the UG, the statement should be "WEIGHTAGE should be an integer between 0 and 100 inclusive." It is actually intended that TAs can add assessments of 0% weightage to give them the flexibility to track aspects of their students' performance that are not counted toward the final mark. For example, the TAs can track their students' marks for homework that is not part of the overall grade.

We argue that this bug is of severity.VeryLow as the user is able to use the command as intended and the message in the app indicates the valid range for WEIGHTAGE.

Items for the Tester to Verify

:question: Issue type

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: Since VeryLow should only be reserved for cosmetic or UI related issues, since this has an effect on the advised behaviour, I believe it should be a Low