Skip to content

refactor(agentic): burst phase starts by default - #21

Merged
cquil11 merged 1 commit into
cquil11/global-system-idle-capfrom
cquil11/burst-phase-start-default
Jul 28, 2026
Merged

refactor(agentic): burst phase starts by default#21
cquil11 merged 1 commit into
cquil11/global-system-idle-capfrom
cquil11/burst-phase-start-default

Conversation

@cquil11

@cquil11 cquil11 commented Jul 28, 2026

Copy link
Copy Markdown

Summary

  • Make Agentic Replay burst warmup and profiling phase starts by default so configured concurrency becomes active within the benchmark window.
  • Preserve recorded inter-turn timing after the initial phase dispatch.
  • Keep spread phase starts available by setting burst_phase_starts to false.
  • Regenerate the CLI option documentation.

Why

Long recorded offsets can otherwise delay initial requests beyond the benchmark warmup or profiling window, preventing the requested concurrency from becoming active.

Validation

  • uv run pytest tests/unit/timing/strategies/test_agentic_replay.py tests/unit/common/config -q (758 passed)
  • uv run pre-commit run
  • uv run ruff check ...
  • uv run ruff format --check ...
  • make generate-cli-docs

Consumed by SemiAnalysisAI/InferenceX#2375.

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

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

Last updated for commit: fb39d0dBrowse 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 marked this pull request as ready for review July 28, 2026 04:08
@cquil11
cquil11 merged commit 8c7a7c7 into cquil11/global-system-idle-cap Jul 28, 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