Skip to content

revert(agentic): restore spread phase starts - #29

Merged
cquil11 merged 1 commit into
cquil11/aiperf-agentx-v1.0from
cquil11/revert-burst-phase-default
Jul 30, 2026
Merged

revert(agentic): restore spread phase starts#29
cquil11 merged 1 commit into
cquil11/aiperf-agentx-v1.0from
cquil11/revert-burst-phase-default

Conversation

@cquil11

@cquil11 cquil11 commented Jul 30, 2026

Copy link
Copy Markdown

Summary

  • Restore spread warmup and profiling phase starts as the Agentic Replay default.
  • Keep --burst-phase-starts available as an explicit throughput-oriented opt-in.
  • Preserve the end-to-start timing cache invalidation from refactor(agentic): burst phase starts by default #22.

Why

The default change in #22 altered trace-replay semantics. Faithful recorded phase offsets should remain the default; benchmarks that want synchronized startup can opt in explicitly.

Validation

  • 759 passed for tests/unit/timing/strategies/test_agentic_replay.py and tests/unit/common/config
  • Pre-commit hooks passed, including test-imports with the worktree source path
  • CLI option documentation regenerated

Signed-off-by: Cam Quilici <cjquilici@gmail.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@d4b2362b151d5d2f422390e0ce3f2fa1ebc22ac5

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@d4b2362b151d5d2f422390e0ce3f2fa1ebc22ac5

Last updated for commit: d4b2362Browse code

@github-actions

Copy link
Copy Markdown

Fern Docs Preview: generation failed — see the Actions log for details. This does not block merge; ask a maintainer to retry if needed.

@cquil11
cquil11 merged commit 6cd93f0 into cquil11/aiperf-agentx-v1.0 Jul 30, 2026
9 of 21 checks passed
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