owenl131 / pe

0 stars 0 forks source link

MA1511 and MA1512 unable to allocate #5

Open owenl131 opened 1 year ago

owenl131 commented 1 year ago

When MA1511 and MA1512 are added without selecting any slots, the system is unable to allocate slots for them. This affects usability because those are core pre-allocated mods for year 1 engineering students.

image.png

On the other hand, a valid allocation does exist:

image.png

soc-pe-bot commented 1 year ago

Team's Response

The dev team acknowledges that this is an issue, however, as the users are notified of the lessons which have not been allocated, they still have the option to set the lesson and hence operation of the application is not entirely compromised, hence we feel that it should be set to a severity of low.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: Team should accept this bug as an indication that the algorithmic correctness of the permutation generation code is incorrect. In particular, in duke.timetable.TimetableDict, line 395, assigning the class numbers in running order will not generate all permutations.