search

AY2425S1-CS2103T-T08-4 / tp

MIT License
0 stars 5 forks source link

[PE-D][Tester A] aggGrade feature not meaningfully useful #216

Closed nus-pe-bot closed 1 day ago

nus-pe-bot commented 4 days ago

image.png

Suppose I am a TA that wishes to find the mean score of the test named "finals" of the course CS1231S. If course I would do the command aggGrade mean n/finals, as per the User Guide. But due to the course CS1231S and the course CS1101S having 2 tests both named finals, the mean score is computed with the finals scores of CS1101S as well. This creates a mean score calculatesd using the finals scores of both mods CS1101S and CS1231S, which does not give any meaningful result, as a TA, I would find this extremely inconvenient. One possible fix would be to aggregate Test Name with Course, so such issues do not occur :)

Labels: type.FeatureFlaw severity.High original: FlyingDonkeys/ped#4

Oceankoh commented 4 days ago

Hi, thanks for the report. Indeed, we did not account for users who may be TAs of multiple mods when writing the aggGrade function. However, this issue only affects a small minority of users since most TAs only in charge of a single mod.

Thus we would assign this bug a severity of medium.