Skip to content

Commit 61a64b7

Browse files
joseph-isaacsclaude
andcommitted
Use a 5 s random-access measurement time limit
A 10 s control run on this branch moved by the same amount against develop as the 5 s run, so halving the time limit does not change the results; it only halves the benchmark step. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UeniB9mYWJs2z49XPgvbPi Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
1 parent 11b301f commit 61a64b7

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

‎scripts/random-access-split.py‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@
2424
LOCAL_ONLY_FORMATS = {"arrow-ipc", "litdata"}
2525
PATTERNS = ["correlated", "uniform"]
2626
OPEN_MODES = ["cached", "reopen"]
27+
# Seconds each measurement runs for, passed as --time-limit.
28+
TIME_LIMIT_SECS = 5
2729

2830

2931
def drop_os_caches() -> None:
@@ -98,6 +100,8 @@ def run_combinations(emit_ingest_records: bool, remote_data_dir: str | None) ->
98100
pattern,
99101
"--open-mode",
100102
open_mode,
103+
"--time-limit",
104+
str(TIME_LIMIT_SECS),
101105
"-d",
102106
"gh-json",
103107
"-o",

0 commit comments

Comments
 (0)