Skip to content

npu 910b4: ModelArts bring-up fixes (visible NPU id, CANN 8.x, Ray NPU visibility, pin waiver) - #227

Merged
michaellchung merged 1 commit into
mainfrom
s20-ascend-npu
Oct 10, 2026
Merged

michaellchung merged 1 commit into
mainfrom
s20-ascend-npu

Conversation

@michaellchung

Copy link
Copy Markdown
Contributor

Found while running yeto's verl island on a Huawei ModelArts notebook (1x910B4, CANN 8.5.1, verl fork acad9875, vllm-ascend 0.16). Do not merge without review.

  • catalog._first_npu_id(): npu-smi board query used ID 0; the container only sees physical ID 4.
  • CANN 8.x version file: <arch>-linux/ascend_toolkit_install.info.
  • verl_main: on Ascend nodes set RAY_EXPERIMENTAL_NOSET_ASCEND_RT_VISIBLE_DEVICES=1; otherwise the CPU-only YetoTaskRunner sees no NPU and verl refuses vLLM-Ascend. Verified on the card through the env var; the code path itself has not run on a card yet.
  • YETO_VERL_ALLOW_UNPINNED=1 (bring-up only): pin mismatches recorded as problems_waived instead of exit 2.

On-card result with these workarounds: 20 rounds Qwen3-0.6B LoRA gsm8k, exit 0, 21/21 publication checks VERIFIED, abs_diff 0.008-0.013, k3 0.0003-0.0006, tis_clipfrac 0. Evidence: s1-runs/s20-ascend-rl-a-20261010/.

Tests: tests/test_npu_910b4_prep.py, test_hw_compat_group.py, test_accel.py (68 passed); verl adapter tests (62 passed).

🤖 Generated with Claude Code

… file, Ray NPU visibility, pin waiver)

Found on a ModelArts notebook (1x910B4, CANN 8.5.1, verl fork acad9875):
- npu-smi board query used ID 0; the container only sees physical ID 4.
- CANN 8.x keeps its version in <arch>-linux/ascend_toolkit_install.info.
- Ray hides NPUs from CPU-only actors; verl then refuses vLLM-Ascend in
  YetoTaskRunner. Set RAY_EXPERIMENTAL_NOSET_ASCEND_RT_VISIBLE_DEVICES=1
  on Ascend nodes only.
- YETO_VERL_ALLOW_UNPINNED=1 (bring-up only) records pin mismatches as
  problems_waived instead of exit 2.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@michaellchung
michaellchung merged commit aa4c804 into main Oct 10, 2026
1 of 3 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.

2 participants