nusCS2113-AY1920S1 / pe-dev-response

0 stars 0 forks source link

Cap overall takes in the same module code and grade twice to give another cap value #1023

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Image 087.jpg You can see the difference in the cap from adding the same module and grade entry twice vs only one module and grade entry In this case the module cg2023 is being added to cap overall twice with the same grade f


[original: nusCS2113-AY1920S1/pe-interim#1023]

andrewleow97 commented 4 years ago

Team's Response

Firstly, the cap overall command functions the same way as other cap calculators available around, it differs only that you only need enter the module code and the letter grade obtained, and it automatically checks the MCs and the CAP and calculates it for you. This is as most people would remember the module code and the letter grade obtained more easily as compared to how many MCs the module has as well as the decimal conversion of the letter grade they obtained.

The assumption is that the user wouldn't input the same module/grade twice, similarly to how you wouldn't calculate your CAP with the same result repeated when doing it manually. Thus this is not a bug and is rejected.

It is a suggestion that we can take into consideration to remove duplicates when calculating one's cap, and will be taken into consideration

Downgrading this to severity.low as it doesn't break the functionality, and only inconveniences those who double input the same module for calculation for some reason.

Duplicate status (if any):

--