Skip to content

Invalidate stale memory-mapped dataset caches - #56

Closed
hbarclay wants to merge 1 commit into
masterfrom
fix/bump-mmap-manifest-version
Closed

hbarclay wants to merge 1 commit into
masterfrom
fix/bump-mmap-manifest-version

Conversation

@hbarclay

Copy link
Copy Markdown

Older versions applied --trace-idle-gap-cap-seconds directly to the turn timestamps when building the memory-mapped dataset cache. That behaviour was removed in favour of applying the cap at runtime, but MANIFEST_VERSION was not bumped, so hosts that still hold a cache built by an older version keep replaying the old capped timeline and produce different results than a fresh host. Bumping the manifest version forces those caches to be rebuilt.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Try out this PR

Quick install:

pip install --upgrade --force-reinstall git+https://github.com/ai-dynamo/aiperf.git@4bba01b21d0d7067614dcfd0f25bba3c34d966bc

Recommended with virtual environment (using uv):

uv venv --python 3.12 && source .venv/bin/activate
uv pip install --upgrade --force-reinstall git+https://github.com/ai-dynamo/aiperf.git@4bba01b21d0d7067614dcfd0f25bba3c34d966bc

Last updated for commit: 4bba01b • Browse code

@hbarclay

Copy link
Copy Markdown
Author

Superseded by keying the cache on the harness commit from the launcher.

@hbarclay hbarclay closed this Oct 19, 2026
@hbarclay
hbarclay deleted the fix/bump-mmap-manifest-version branch October 19, 2026 02:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant