Skip to content

Commit

Permalink
Remove old workaround
Browse files Browse the repository at this point in the history
no longer grow the number of games per task,
it is no longer needed for fishtest performance
and limits the number of workers that can be assigned e.g. to a progression test.
  • Loading branch information
vondele authored and ppigazzini committed Feb 4, 2025
1 parent 2a4e69b commit c7e0903
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions server/fishtest/rundb.py
Original file line number Diff line number Diff line change
Expand Up @@ -873,10 +873,7 @@ def worker_cap(self, run, worker_info):
game_time = estimate_game_duration(run["args"]["tc"])
concurrency = worker_info["concurrency"] // run["args"]["threads"]
assert concurrency >= 1
# as we have more tasks done (>250), make them longer to avoid
# having many tasks in long running tests
scale_duration = 1 + min(4, len(run["tasks"]) // 250)
games = self.task_duration * scale_duration / game_time * concurrency
games = self.task_duration / game_time * concurrency
if "sprt" in run["args"]:
batch_size = 2 * run["args"]["sprt"].get("batch_size", 1)
games = max(batch_size, batch_size * int(games / batch_size + 1 / 2))
Expand Down

0 comments on commit c7e0903

Please sign in to comment.