Skip to content
Snippets Groups Projects

Improve wait time

Merged
Taico Aertsrequested to merge
wait-time-improvements into development
All threads resolved!

Queue normally shows an average wait time over the last hour. The main problem is that an average over the last hour, for a lab that takes 2-4 hours, is almost always off from what students truly need to wait. Instead, a better predictor for wait time is the time the oldest pending request has been waiting for.

This MR adds this current wait time. It is defined as the wait time of the oldest pending request. If no such request exists, it uses the wait time that was experienced for the last handled request.

The calculation of the current wait time also works with timeslotted labs. Here the start of the timeslot is used rather than the creation time of the request.

Of course, current wait time is also not perfect. Because it is not an average, the value is expected to fluctuate a lot more. Prioritization of requests also messes with the current wait time. Therefore, both values are displayed.

Edited by Taico Aerts

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Taico Aerts marked this merge request as ready

    marked this merge request as ready

  • Taico Aerts
  • Taico Aerts requested review from @rbackx and @cedricwilleken

    requested review from @rbackx and @cedricwilleken

  • Taico Aerts resolved all threads

    resolved all threads

  • Taico Aerts resolved all threads

    resolved all threads

  • Taico Aerts added 1 commit

    added 1 commit

    Compare with previous version

  • Taico Aerts
  • Taico Aerts
  • Taico Aerts resolved all threads

    resolved all threads

  • Taico Aerts resolved all threads

    resolved all threads

  • Taico Aerts added 1 commit

    added 1 commit

    Compare with previous version

  • Ruben Backx approved this merge request

    approved this merge request

  • Taico Aerts started a merge train

    started a merge train

  • Taico Aerts mentioned in commit 0033502b

    mentioned in commit 0033502b

  • merged

  • Please register or sign in to reply
    Loading