center-randomize icon indicating copy to clipboard operation
center-randomize copied to clipboard

Auto rerun if all students are not allocated

Open B-kash opened this issue 10 months ago • 2 comments

Summary

On the readme file it says

Run the program and re-run if

  • Any school has students who have not been assigned a center.

This feels very weird.

It would be very userfriendly if all the students are allocated on the single go.

Proposed solution

Keep the list of students that are not allocated and try allocating them if they are not allocated at the end of the run.

B-kash avatar Apr 26 '24 09:04 B-kash

The way this is implemented, allocation of all students cannot be guaranteed. However we may re run the app until all students are allocated. We would also need to set a max retry count (say 10) and show error message if it fails more than that. In such case we need to revisit to see if more centers need to be added or more capacity is needed

sumanashrestha avatar May 05 '24 13:05 sumanashrestha

I have created a PR with the changes. This should rerun default 10 times if the allocation failed and also the retries can be updated with the arguments.

B-kash avatar May 13 '24 20:05 B-kash