mazfil / lab-allocator

COMP3500 - Resource maximisation planning system for labs
3 stars 1 forks source link

Backend 87&94 #101

Closed Hexuan-Meng closed 2 months ago

Hexuan-Meng commented 3 months ago

The details of implementation is in the comments of issue#87 and issue#94, and the commit message.

The backend algorithm now works correctly, and we have a new fitness function to count for parallel classes. I believe there are some wrong calculations in other existing fitness functions. I can fix them later if needed.

I also encountered a 'quota exceeded' error for firebase when running the backend solver. Not sure what's happened as I can run it last week without error.

alexdboxall commented 2 months ago

Looks good to me - thanks for fixing those bugs and adding the new fitness function! The firebase quota exceeded I think happened because of front end testing that was happening at the same time, so nothing to worry about there.

@matthewcawley02 after you've reviewed it, it should be good to merge

matthewcawley02 commented 2 months ago

Good job, looks good to me.