search

AY2425S1-CS2103T-T08-4 / tp

MIT License
0 stars 5 forks source link

[PE-D][Tester F] The function `aggGrade median` works for current screen but does not work as intended #241

Closed soc-se-bot closed 2 days ago

soc-se-bot commented 5 days ago

Can possibly check if there is an issue with the calculation of median within the current screen Picture12.png

Picture13.png

Labels: severity.Low type.FunctionalityBug original: testing1234567891011121314/ped#9

izruff commented 3 days ago

Hi, thanks for reporting. Looking at the screenshots, the result of aggGrade median is correct. The median is taken from the overall scores of each person in the displayed list: 100.00, 4.62, 0.00, 0.00. Therefore the median is (4.62+0.00)/2 = 2.31, as shown by the output message.

Based on this, we do not consider this as a functionality bug.

However, we realize that the user guide does not specifically mention how the command treats difference in grade lists and whether a person with no grades listed are counted into the calculation. This is indeed a valid question, and we will consider adding an example of how the aggregate function is being calculated in the user guide.

juancarlovieri commented 3 days ago

@dominopizzaaaa consider adding this to UG