center-randomize
center-randomize copied to clipboard
[Enhancement] ✨ Add retry mechanism if all students are not allocated
Summary
Add retry mechanism incase of remaining students allocation
Pull request type
Please try to limit your pull request to one type, and submit multiple pull requests if needed.
Please tick the type of change your PR introduces:
- [ ] Bugfix
- [x] Feature
- [x] Code style update (formatting, renaming)
- [x] Refactoring (no functional changes, no API changes)
- [ ] Build related changes
- [ ] Documentation content changes
- [ ] Other (please describe):
What is the current behaviour?
Please describe the behaviour that needs to be modified or linked to a relevant issue.
Issue Number: 81
What is the current behaviour?
Please describe the changes or updates being made by this pull request.
- Added a new argument that can be passed called max-retries (Default: 10)
- Rerun the allocation upto max-retries if not successfully allocated all the students
- Small code refactoring changes
Checklist
Please review and complete the following checklist before submitting your pull request:
- [x] I have tested the changes locally and they work as intended.
- [x] I have provided a detailed description of the changes made.
- [x] I have reviewed the formatting and ensured it follows the project's style guidelines.
- [ ] I have assigned the appropriate labels to the pull request.
- [x] I have added necessary documentation or updated existing documentation for the changes made.
- [ ] I have addressed code review feedback and resolved any conflicts with the main branch.
Other information
Include screenshots of the component before and after the change.