moest-np / center-randomize

Script is to assign exam centers to students
MIT License
766 stars 364 forks source link

refactor: assign less than min threshold students to previous center #51

Open ashiishme opened 1 month ago

ashiishme commented 1 month ago

The new implementation fixes the #13 of allocating small amount of students to a new center.

Details

The PR also abstract the center allocation logic to a new function allocate_to_centers.

I saw that there is a WIP PR related to the unit test. I can also add the test for allocate_to_centers function once the that PR is merged to make sure that the allocation is working as expected.

Resolves #13