seriouslia0 / pe

0 stars 0 forks source link

Sort command description in the user guide is not complete. #12

Open seriouslia0 opened 1 year ago

seriouslia0 commented 1 year ago

Screenshot 2023-04-14 at 3.14.11 PM.png

The description of the sort command in the user guide does not state in which order the values are sorted. As a result, the user would not know how to interpret the ordering inside the UI if it is not obvious what the order is, such as when u sort by the number of modular credits

nus-pe-script commented 1 year ago

Team's Response

If you meant whether it is by ascending or descending or some order between each category, we believe the order is arbitrary and does not need to be stated explicitly.

If you meant that within each category how it is sorted, then that is not important for our sort command.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: As i have stated in another bug report.

Specifically, if you intended to sort only by categories, then this is not really 'sorting', but merely filtering. An expected user is likely to adopt the dictionary definition of sorting, which states: "the arrangement of data in a prescribed sequence." This sequence, should have a prescription (i.e. an ordering) and not be completely random. With reference to my initially attached screenshot, the sequence [10,4,5] yields no discernable ordering.

This 'arbitrariness' of ordering, since it goes against expected behaviour from the user, should be mentioned in the user guide (assuming it is intended)