nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Split function does not take into account clashes in timing #1923

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

The split function will allocate members to different facilities with the same start time. This may be problematic and cause clashes in timings.

Capture.PNG

In this example, it could have been optimal to allocate all members to NUS Field 2 instead of allocating some to Court 1.


[original: nus-cs2103-AY2122S1/pe-interim#1979] [original labels: type.FeatureFlaw severity.Medium]

Moley456 commented 3 years ago

Team's Response

Not sure what the problem here is. As shown in the screenshot, the capacity of the first facility is filled with 5 people and so the remaining members all be allocated to NUS Field 2.

Clashes in training timings is not an issue because multiple training facilities can be used at the same time.

Bug rejected as screenshots shown are the intended behavior of the application.

Duplicate status (if any):

--