Skip to content

fix(h3): route video device selection through the backend seam - #134

Closed
localai-bot wants to merge 2 commits into
mainfrom
row/ARCH-ONE-SURFACE
Closed

fix(h3): route video device selection through the backend seam#134
localai-bot wants to merge 2 commits into
mainfrom
row/ARCH-ONE-SURFACE

Conversation

@localai-bot

@localai-bot localai-bot commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

ARCH-ONE-SURFACE H3 follow-up: repair the device-specific references introduced by the ABI v12 video fold.

Change

  • Preserve public ABI device: 0=CPU, 1=CUDA.
  • Validate/map it once with MiniMaxH3VideoDeviceType.
  • Create one queue through GetBackend(device_type) and use that queue's device.
  • Add fold-unit coverage for selectors 0/1 and invalid -1/2.
  • Refresh the owning model/roadmap/spec/public/state records.

RED -> GREEN evidence

  • RED on origin/main: python3 scripts/check-device-leakage.py --report -> DSR 34, kcuda=2, baseline 32, exit 1.
  • GREEN at 5d03059e: DSR 32, kcuda=0, baseline unchanged, no allowlist.
  • python3 tests/scripts/test_device_leakage.py: 25/25 pass.
  • Public-doc checker mutation suite: 41/41 pass.
  • Full staged preflight: exit 0.
  • Full pre-push preflight chained to normal push: exit 0.

Honest pending gate

The from-scratch local CPU build was stopped at the operator's request when the shared filesystem reached 100%; test_minimax_h3_video_fold, H3, C API, and CPU build/tests are NOT claimed locally and must pass GitHub CI before review/merge. No GPU job, download, benchmark, release-row edit, baseline change, or allowlist was made.

Draft remains open for the operator's fresh mutation review.

mudler added 2 commits August 7, 2026 22:48
FOLLOWING_AGENTS_PROTOCOL
Assisted-by: Codex:gpt-5 [Codex]
Keep the public ABI selector at 0=CPU and 1=CUDA while converting it once to vt::DeviceType. The H3 shared layer now creates one queue through GetBackend(device_type), restoring the device-leakage ratchet without an allowlist or baseline change.\n\nHelper row: row/ARCH-ONE-SURFACE (PR #134).\n\nFOLLOWING_AGENTS_PROTOCOL\nAssisted-by: Codex:gpt-5 [Codex]
mudler added a commit that referenced this pull request Aug 7, 2026
Keep the public ABI selector at 0=CPU and 1=CUDA while converting it once to vt::DeviceType. The H3 shared layer now creates one queue through GetBackend(device_type), restoring the device-leakage ratchet without an allowlist or baseline change.

Helper row: row/ARCH-ONE-SURFACE-H3-DEVICE (PR #135; replaces #134).

FOLLOWING_AGENTS_PROTOCOL
Assisted-by: Codex:gpt-5 [Codex]
@localai-bot

Copy link
Copy Markdown
Collaborator Author

Superseded by clean replacement PR #135: #135. PR #134\x27s implementation commit encoded the required trailers as literal backslash-n text; refs were not rewritten or force-pushed. The exact tree diff was reapplied from origin/main and committed with real trailer lines on #135.

@localai-bot localai-bot closed this Aug 7, 2026
richiejp pushed a commit to richiejp/vllm.cpp that referenced this pull request Aug 8, 2026
Replacement for malformed-trailer PR mudler#134 on row/ARCH-ONE-SURFACE-H3-DEVICE.

FOLLOWING_AGENTS_PROTOCOL
Assisted-by: Codex:gpt-5 [Codex]
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.

2 participants