Skip to content

Commit

Permalink
Do not schedule meetings that need a bigger room first
Browse files Browse the repository at this point in the history
Scheduling meetings that need a bigger room first does not seem to yield
great results, because we can end up scheduling meetings that need a large
number of slots last with that approach, and slots may then no longer be
available for them.

This update drops the rule. Meetings that need a bigger number of slots are
scheduled first. Other constraints do not influence the order otherwise.
  • Loading branch information
tidoust committed May 21, 2024
1 parent 6d2223f commit 7fe79f7
Show file tree
Hide file tree
Showing 3 changed files with 320 additions and 330 deletions.
4 changes: 1 addition & 3 deletions test/check-tpac-scheduling.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -164,12 +164,10 @@ _No response_`;
let { errors: warnings } = await validateGrid(project);
warnings = warnings.filter(error => error.severity === 'warning' && error.type === 'times');
assert.deepStrictEqual(stripDetails(warnings), [{
session: 42,
session: 58,
severity: 'warning',
type: 'times',
messages: [
'Session not scheduled on Tuesday (2023-09-12) at 11:30 - 13:00 as requested',
'Session not scheduled on Tuesday (2023-09-12) at 14:30 - 16:30 as requested',
'Session not scheduled on Thursday (2023-09-14) at 17:00 - 18:30 as requested'
]
}]);
Expand Down
Loading

0 comments on commit 7fe79f7

Please sign in to comment.