Skip to content

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

Merged
cquil11 merged 2 commits into
cquil11/aiperf-agentx-v1.0from
cquil11/burst-phase-start-default
Jul 28, 2026
Merged

refactor(agentic): burst phase starts by default#22
cquil11 merged 2 commits into
cquil11/aiperf-agentx-v1.0from
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.
  • Carry forward the end-to-start timing cache invalidation already pinned by InferenceX.

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.

Supersedes #21, which was mistakenly targeted at the already-merged feature branch instead of cquil11/aiperf-agentx-v1.0.

cquil11 added 2 commits July 27, 2026 17:44
Include use_end_to_start_delays in the mmap dataset cache fingerprint so caches created before the timing-semantics fix cannot be reused.

Signed-off-by: Cam Quilici <cjquilici@gmail.com>
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:10
@cquil11
cquil11 merged commit 208125a into cquil11/aiperf-agentx-v1.0 Jul 28, 2026
18 of 42 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