[CI][ROCm] Temporarily skip unsupported HY-V4 initialization

[AndreasKaratzas]

HY-V4's CUDA-only implementation in #54160 raises NotImplementedError: hy_v4 does not yet support ROCm during the two HY initialization cases in AMD build 12635 and build 12653. This temporarily skips those unsupported cases while model enablement in #54405 and its backend dependency #54404 are completed. Remove this guard alongside ROCm model enablement; this CI-only change is separate from those support implementations.

• Skip only HYV4ForCausalLM and HYV4MTPModel initialization on ROCm.

Prepared with AI assistance (OpenAI Codex).

[claude]

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

_{Tip: disable this comment in your organization's Code Review settings.}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Team

Run ID: 359bcde7-a599-4fd1-8b99-1341bfc75197

📥 Commits

Reviewing files that changed from the base of the PR and between d9105ea and 1dd6ab1.

📒 Files selected for processing (1)

• tests/models/test_initialization.py

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Summary

Summary by CodeRabbit

• Tests
  • Updated initialization test coverage to skip certain HyV4 model cases on ROCm pending an upstream change.

Walkthrough

The initialization test now skips HYV4ForCausalLM and HYV4MTPModel on ROCm. Other platforms and model architectures retain the existing initialization behavior.

Changes

ROCm initialization testing

Layer / File(s)	Summary
Add ROCm-specific HYV4 skip tests/models/test_initialization.py	test_can_initialize_large_subset calls pytest.skip for the two HYV4 architectures when current_platform.is_rocm() is true.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 1dd6a

On ROCm, the two unsupported HYV4 initialization checks now report as skipped rather than failing, while other initialization coverage remains unchanged. No merge-blocking risk remains.

Suggested reviewers: hmellor, isotr0py, zjy0516

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the temporary ROCm skip for unsupported HY-V4 initialization tests.
Description check	✅ Passed	The description directly explains the ROCm failure, the limited test skip, and the condition for removing the guard.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[AndreasKaratzas]

/ci run

[github-actions]

✅ Triggered Buildkite CI #87491 for commit 1dd6ab18c36f.