search

JinYixuan-Au / pe

0 stars 0 forks source link

CAP calculator outputs incorrect result #1

Open JinYixuan-Au opened 3 years ago

JinYixuan-Au commented 3 years ago

CAP calcutor outputs 0 for this CS1010E module with A- grade. This is an incorrect value due to our grade information. image.png

nus-pe-bot commented 3 years ago

Team's Response

I am not sure what tester is asking. In fact the outputs for CS1010E with grade A- is shown in the picture with MC of 4 and grade of 4.5 which is correct.

Perhaps the tester is confused about this function. Maybe the tester assumed that when the result displayed the second line about S/U CS1010E will show the tester the CAP of 4 and graded MCs of 4. However in our UG for the Set SU by semester it shows:

image.png

The first sentence shows that modules will be S/U-ed from the lowest grade, and show the updated CAP. Hence the result of 0 CAP and 0 MCs is intended and therefore not a bug at all. This complements the second half of the output, which is left out of the screenshot ,which would have suggested the tester to just S/U his GER1000H.

Under normal use cases, nobody will SU a higher grade when a lower grade is available to be SU-ed.

For these reasons, that it is an intended functionality and that it is already stated in the UG about how this function works, we will be rejecting this bug.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I basically understand the reason for this S/U result, but I still think it is vague to output 0 in your app. It should output the real reason in this situation. Also, this situation is not explained clearly in your UG.

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]