Skip to content

[VideoGenX] run native H3 sweeps from YAML recipes / [VideoGenX] 使用 YAML 配方运行原生 H3 扫描 - #2916

Open
edwingao28 wants to merge 1 commit into
feat/videogenx-gpu-evidencefrom
feat/h3-cross-hardware
Open

edwingao28 wants to merge 1 commit into
feat/videogenx-gpu-evidencefrom
feat/h3-cross-hardware

Conversation

@edwingao28

@edwingao28 edwingao28 commented Sep 9, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Changes

  • Add native H3 video sweeps for SGLang on H100/H200/B200 and vLLM-Omni on H200. Configuration support does not qualify an untested row.
  • Bind workload, model snapshot, image and engine settings into the service/client recipe. Reject model, port and GPU-count mismatches before submission; disable conditioning-cache reuse across requests.
  • Bundle delivered media, client timing windows, serving GPU UUIDs, scheduler allocation and power. Keep executed checkout separate from the Actions head.
  • Wait for video completion before statistics. Require every selected cell for full-sweep reuse; retain failures. Limit native jobs to 75 minutes and workflows to 90 minutes, without automatic retry.
flowchart LR
    A["Workload + engine recipe"] --> B["Native server and C1 client"]
    B --> C["Media + request windows"]
    B --> D["GPU UUIDs + allocation"]
    C --> E["Media and power checks"]
    D --> E
    E --> F["Sealed artifacts"]
    F --> G["Statistics"]
Loading

Test setup

  • Official SGLang 797af717 image pinned by digest; H3 model 42ed227e, BF16, TP2/Ulysses2, h3-v0.1/release-v1.
  • H100, concurrency 1: 20 measured requests + one disjoint client warmup; four participating GPUs, eight allocated GPUs, 192 CPUs.
  • Output: 124 frames at 24 fps, 1344×768, 32 kHz stereo. Requested duration is 5 s; decoded duration is 124/24 s.
  • C1: submission through downloaded media, 0.5 s polling; excludes subsequent local validation.

Results

Testing: Targeted H100 E2E, attempt 1 passed.

Runtime / original run GPU Measured + warmup C1 p50 Status
797af717 / 41604835531 H100 20 + 1 73.8847 s Targeted row passed
v0.5.23 / 41280191475 H100 20 + 1 73.8835 s Historical row passed
v0.5.23 / 41280191475 B200 20 + 1 35.6733 s Historical row passed
  • New H100 audit: six original ZIP digests, 56 file seals, 21 complete audio/video decodes, zero request failures.
  • GPU power: four matching UUIDs; 6,604 raw rows pass window and energy checks. Statistics reports 1/1 after video completion.
  • One run, no repeated matched A/B trials: the H100 medians do not establish a speedup or equivalence. Historical B200 data does not qualify 797af717 on B200; these rows are not a scaling curve.

Validation scope

  • Coverage: the accepted run is the H100 row. Current-runtime H200/B200 and the full matrix are not covered; semantic quality is unjudged.
  • GitHub reports a merge conflict; benchmark results have not been published.
  • Actual per-rank 49-update/50-sigma traces, complete installed-runtime inventory and DCGM exporter content digest remain unverified.
  • Auxiliary host exporter failed with TaskProlog 127; required GPU-power checks passed separately.
Validation details
PR checklist and change type

Type of Change

  • Bug fix
  • New feature
  • Configuration change
  • Documentation update
  • Other (please describe)

Checklist

  • I have tested my changes locally
  • I have updated documentation if necessary
  • If python -m infx.matrix.changed --base origin/main lists affected config keys, I have appended one new entry covering them to the physical end of inferencex-e2e/perf-changelog.yaml; if it lists none, I have added none. I have not edited historical entries
  • If this PR appends an inferencex-e2e/perf-changelog.yaml entry, it carries exactly one primary sweep label (a maintainer applies it on fork PRs): full-sweep-fail-fast (recommended), full-sweep-enabled, or non-canary-full-sweep-enabled. Optional modifiers all-evals, evals-only, and agentx-fast require a primary label; the last two block reuse while applied.
  • Before merging via reuse, an authorized maintainer (OWNER/MEMBER/COLLABORATOR) has commented /use <run_id> (or the legacy /reuse-sweep-run) on this PR. Do this only once there is a final full sweep that is all green with evals passing, since after this comment the primary sweep label will no longer automatically kick off new sweeps. Remove and re-add the primary sweep label to force a new sweep.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Thanks for the contribution! Please reach out to respective companies' CODEOWNER to fill in the latest PR_REVIEW_CHECKLIST.md before pinging core maintainer on Slack for review. In order for the signoff PR check bot to trigger, you must follow the PR_REVIEW_CHECKLIST.md template correctly, including the phrase As a PR reviewer and CODEOWNER, I have reviewed this and have.

For PR verification, add the full-sweep-fail-fast label (strongly recommended) to this PR — the benchmark sweep only runs on labeled PRs. Use full-sweep-enabled only if you need matrix jobs to keep running past a failure.

PR authors are responsible for ensuring that after merging, all GitHub Action jobs fully pass. A lot of the time, failures are just flakes and simply re-running the failed jobs will fix it. See GitHub's docs on re-running failed jobs


感谢你的贡献!请联系相应公司的 CODEOWNER 填写最新的 PR_REVIEW_CHECKLIST.md,然后再在 Slack 上联系核心维护者进行审阅。为了触发 signoff PR 检查机器人,你必须正确遵循 PR_REVIEW_CHECKLIST.md 模板,包括保留英文语句 As a PR reviewer and CODEOWNER, I have reviewed this and have。

如需进行 PR 验证,请为此 PR 添加 full-sweep-fail-fast 标签(强烈推荐)— 基准测试 sweep 仅在带有标签的 PR 上运行。仅当需要矩阵任务在失败后继续运行时才使用 full-sweep-enabled。

PR 作者有责任确保合并后所有 GitHub Action 任务完全通过。 很多时候失败只是偶发抖动(flake),重新运行失败的任务即可解决。参见 GitHub 关于重新运行失败任务的文档

@edwingao28
edwingao28 force-pushed the feat/h3-cross-hardware branch from a7facf5 to 7019bb2 Compare September 9, 2026 08:42
@edwingao28 edwingao28 changed the title [H3 CI] add hardware sites and serving matrices / 接入硬件站点与服务测量 [H3 CI] compare frozen hardware runs and retain serving evidence / 保留跨硬件服务证据 Sep 9, 2026
@edwingao28
edwingao28 force-pushed the feat/h3-cross-hardware branch from 2327589 to 8247212 Compare September 27, 2026 06:32
@edwingao28 edwingao28 changed the title [H3 CI] compare frozen hardware runs and retain serving evidence / 保留跨硬件服务证据 [H3] integrate manual video benchmarks with the current infx planner / [H3] 将手动视频基准测试接入当前 infx planner Sep 27, 2026
@edwingao28
edwingao28 changed the base branch from feat/h3-video-ci to main September 27, 2026 06:33
@functionstackx

Copy link
Copy Markdown
Collaborator

Sorry, over the weekend, there was 2 major refactors to clean up the technical debt accumalated over the past 11 months of moving at the speed of light. We don't see any major refactors in the forthseeable future besides cleaning up AMD multinode AgentX pile of bash. As much, due to the refactors, u would need to ask your agent to rebase from remote main@latest. Thank you in advance for ur understanding

cursor Bot pushed a commit that referenced this pull request Sep 28, 2026
补充 #2916 与 App #1193 的契约矩阵、CPU 验证结果与下一步决策说明。

Co-authored-by: Wenyao Gao <edwingao28@users.noreply.github.com>
@edwingao28 edwingao28 changed the title [H3] integrate manual video benchmarks with the current infx planner / [H3] 将手动视频基准测试接入当前 infx planner [Video] integrate manual video benchmarks with the current infx planner / 将手动视频基准测试接入当前 infx planner Oct 5, 2026
edwingao28 added a commit that referenced this pull request Oct 17, 2026
…aluator

Carries the MiniMax-H3 serving harness from #2916 onto main. The evaluator
gains srt-serve/srt-client/srt-finalize so a generic srt-slurm service runs
the diffusion server, each concurrency publishes its own measurement window,
and the dashboard bundle is assembled from the native power package.

中文:将 #2916 的 MiniMax-H3 serving harness 迁移到 main。evaluator 新增
srt-serve/srt-client/srt-finalize:由 srt-slurm generic service 启动 diffusion
server,每个并发度发布自己的测量窗口,并基于原生功耗包组装仪表板产物。
@edwingao28
edwingao28 force-pushed the feat/h3-cross-hardware branch 5 times, most recently from 4aa0638 to ddc4127 Compare October 17, 2026 04:18
edwingao28 added a commit that referenced this pull request Oct 17, 2026
…aluator

Carries the MiniMax-H3 serving harness from #2916 onto main. The evaluator
gains srt-serve/srt-client/srt-finalize so a generic srt-slurm service runs
the diffusion server, each concurrency publishes its own measurement window,
and the dashboard bundle is assembled from the native power package.

中文:将 #2916 的 MiniMax-H3 serving harness 迁移到 main。evaluator 新增
srt-serve/srt-client/srt-finalize:由 srt-slurm generic service 启动 diffusion
server,每个并发度发布自己的测量窗口,并基于原生功耗包组装仪表板产物。
@edwingao28
edwingao28 force-pushed the feat/h3-cross-hardware branch from ddc4127 to f7c7f58 Compare October 17, 2026 04:31
edwingao28 added a commit that referenced this pull request Oct 17, 2026
…aluator

Carries the MiniMax-H3 serving harness from #2916 onto main. The evaluator
gains srt-serve/srt-client/srt-finalize so a generic srt-slurm service runs
the diffusion server, each concurrency publishes its own measurement window,
and the dashboard bundle is assembled from the native power package.

中文:将 #2916 的 MiniMax-H3 serving harness 迁移到 main。evaluator 新增
srt-serve/srt-client/srt-finalize:由 srt-slurm generic service 启动 diffusion
server,每个并发度发布自己的测量窗口,并基于原生功耗包组装仪表板产物。
@edwingao28
edwingao28 force-pushed the feat/h3-cross-hardware branch 2 times, most recently from 068d64f to 0cb4c84 Compare October 17, 2026 04:45
@edwingao28 edwingao28 changed the title [Video] integrate manual video benchmarks with the current infx planner / 将手动视频基准测试接入当前 infx planner [VideoGenX] run H3 video generation through the native srt-slurm launcher / 通过原生 srt-slurm launcher 运行 H3 视频生成 Oct 17, 2026
@edwingao28
edwingao28 marked this pull request as ready for review October 17, 2026 06:44
@edwingao28
edwingao28 requested a review from a team October 17, 2026 06:44
@edwingao28 edwingao28 removed the full-sweep-fail-fast Full sweep with canary gate; first failure cancels the rest of that matrix (recommended) label Oct 29, 2026
@edwingao28
edwingao28 force-pushed the feat/h3-cross-hardware branch from 2c603bb to 5e2556d Compare October 29, 2026 20:11
@edwingao28
edwingao28 changed the base branch from main to feat/videogenx-gpu-evidence October 29, 2026 20:11
@edwingao28
edwingao28 marked this pull request as draft October 29, 2026 20:12
@edwingao28 edwingao28 changed the title [VideoGenX] run H3 through native sweeps with measured power [VideoGenX] run native H3 sweeps from YAML recipes / [VideoGenX] 使用 YAML 配方运行原生 H3 扫描 Oct 29, 2026
@edwingao28
edwingao28 added this pull request to stack #4660 October 29, 2026 20:20
@edwingao28
edwingao28 force-pushed the feat/h3-cross-hardware branch from 5e2556d to 6101fe0 Compare October 29, 2026 22:03
@edwingao28
edwingao28 force-pushed the feat/h3-cross-hardware branch from 6101fe0 to 867ad47 Compare October 29, 2026 22:13
@edwingao28
edwingao28 force-pushed the feat/h3-cross-hardware branch from 867ad47 to 5934a67 Compare October 30, 2026 20:25
@edwingao28
edwingao28 force-pushed the feat/h3-cross-hardware branch 2 times, most recently from 9752a29 to d142e58 Compare November 1, 2026 01:14
@edwingao28
edwingao28 force-pushed the feat/h3-cross-hardware branch 2 times, most recently from 5c99c04 to b8ba4cf Compare November 1, 2026 06:16
@edwingao28 edwingao28 added the full-sweep-fail-fast Full sweep with canary gate; first failure cancels the rest of that matrix (recommended) label Nov 1, 2026 — with ChatGPT Codex Connector
@edwingao28
edwingao28 force-pushed the feat/h3-cross-hardware branch 3 times, most recently from b7e4847 to 6abe3b6 Compare November 3, 2026 09:46

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full-sweep-fail-fast Full sweep with canary gate; first failure cancels the rest of that matrix (recommended)

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants