Skip to content

feat(inferencex_atom): add M4 vLLM parity suite and W1 variant#249

Draft
hnimra-amd wants to merge 20 commits into
hnimrama/ix-atom-multinodefrom
hnimrama/AIMVT-250-add-vllm-parity
Draft

feat(inferencex_atom): add M4 vLLM parity suite and W1 variant#249
hnimra-amd wants to merge 20 commits into
hnimrama/ix-atom-multinodefrom
hnimrama/AIMVT-250-add-vllm-parity

Conversation

@hnimra-amd

@hnimra-amd hnimra-amd commented Jul 8, 2026

Copy link
Copy Markdown

@hnimra-amd
hnimra-amd marked this pull request as draft July 8, 2026 19:26
…tric

Extend the variant schema with nnodes, pipeline parallelism, rendezvous
addresses, and optional single-node throughput baseline for
scaling.efficiency_pct. Multinode sweep cells include PP and NNODES in
the threshold cell key.
When params.nnodes is greater than one, launch one server per host with
per-rank distributed executor flags, poll all ranks for failures, and
run the benchmark client on the head node only. Single-node runs are
unchanged.
…riant

Ship example cluster JSON, config, and seeded thresholds for the W1
DeepSeek R1 FP8 multinode reference cell. Thresholds stay record-only
until lab confirmation.
Extend FakeOrch with exec_on_head and per-host exec tracking; add unit
tests for distributed server launch, head-only client, scaling metric,
and perf_multi variant schema.
Add lab run instructions for the 2-node W1 perf_multi stem and describe
nnodes, pipeline parallelism, rendezvous, and scaling baseline fields.
Multinode container runs need in-container sshd on port 2224; some ROCm
images ship without openssh-server. Fall back to apt install before start.
…cycle

The function body was left orphaned after the du_bytes import, causing an IndentationError when pytest collected inferencex_atom_single.
… PP flags

Replace unrecognized vLLM distributed CLI args with ATOM SPMD env and -dp when TP allows; TP8 two-node runs independent replicas per host. Harden multinode sshd setup (iproute2, pgrep fallback) and health-check all ranks.
…and report

Enable threshold enforcement for the 2-node perf variant, gate scaling.efficiency_pct as a dedicated metric tier, and surface it in the IX Run Deck preset.
Restore the newline between the docstring and timing call that was lost during rebase conflict resolution.
Collection failures emit CollectReport objects without user_properties; avoid AttributeError so pytest-html surfaces the real collection error.
Drop the misleading _single suffix so multinode runs use cvs run inferencex_atom; keep inferencex_atom_single as a deprecated config framework alias.
Clarify lab setup order so Makefile can manage .cvs_venv without an active shell inside it.
Apply ruff formatting and resolve lint issues in the 12 multinode-touched Python files so fmt-check and ruff check pass on this branch delta.
@hnimra-amd
hnimra-amd force-pushed the hnimrama/ix-atom-multinode branch from 3117e6d to 33b0e4a Compare July 13, 2026 18:24
…riant

Add mi355x_atom_multi cluster file, perf_multi config/threshold scaffold, README runbook, and config-loader unit test mirroring the MI300X 2-node path.
Register inferencex_atom_vllm_single with compare.vllm metrics, IX Run Deck preset, and MI300X DeepSeek W1 parity config/threshold sibling to ATOM perf.
@hnimra-amd
hnimra-amd force-pushed the hnimrama/AIMVT-250-add-vllm-parity branch from 10e4314 to b1b765b Compare July 13, 2026 19:05
Ship MI300X/MI355X vllm_perf_multi configs and thresholds so vLLM parity matches ATOM multinode sweep coverage.
@hnimra-amd
hnimra-amd force-pushed the hnimrama/ix-atom-multinode branch 2 times, most recently from b11c5d6 to b71e9d8 Compare July 21, 2026 21:27
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