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

[Enhancement] ✨ Add retry mechanism if all students are not allocated

Open B-kash opened this issue 9 months ago • 1 comments

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.

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