[https://nvbugs/5814504][fix] Add skip_pre_hopper flag on NVILA & Nano V2 VLMs

[yechank-nvidia]

Summary by CodeRabbit

• Tests
  • Updated test configurations for multimodal LLM functionality to optimize execution across different hardware environments.

[yechank-nvidia]

/bot run

[coderabbitai]

📝 Walkthrough

Walkthrough

Two test classes in the multimodal LLM API test suite are annotated with the @skip_pre_hopper decorator to conditionally skip these tests in pre-hopper environments. No test logic or functionality is modified.

Changes

Cohort / File(s)	Summary
Test Skipping Configuration tests/integration/defs/accuracy/test_llm_api_pytorch_multimodal.py	Added @skip_pre_hopper decorator to TestNVILA_8B and TestNemotron_Nano_12B_V2_VL test classes to skip execution in pre-hopper environments.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description is entirely missing; only the placeholder '@coderabbitai summary' is present with no explanation of the change, rationale, or test coverage details.	Add a complete description explaining what the skip_pre_hopper flag does, why these specific tests need it, and confirm test coverage is adequate for the changes.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The PR title accurately describes the main change: adding @skip_pre_hopper decorators to two test classes (NVILA_8B and Nemotron_Nano_12B_V2_VL).

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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 and usage tips.}

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

tests/integration/defs/accuracy/test_llm_api_pytorch_multimodal.py (1)

1-1: ⚠️ Potential issue | 🟠 Major

Add the required NVIDIA copyright header.

Line 1: this file has been modified but still lacks the required NVIDIA copyright header with the latest modification year. Please add the repo-standard header (updated to 2026) at the top of the file. As per coding guidelines, “All TensorRT-LLM source files (.cpp, .h, .cu, .py, and other source files) should contain an NVIDIA copyright header with the year of latest meaningful modification”.

[tensorrt-cicd]

PR_Github #34759 [ run ] triggered by Bot. Commit: f098163

[tensorrt-cicd]

PR_Github #34790 [ run ] triggered by Bot. Commit: f098163

[tensorrt-cicd]

PR_Github #34790 [ run ] completed with state SUCCESS. Commit: f098163
/LLM/release-1.2/L0_MergeRequest_PR pipeline #319 completed with status: 'FAILURE'

⚠️ Action Required:

• Please check the failed tests and fix your PR
• If you cannot view the failures, ask the CI triggerer to share details
• Once fixed, request an NVIDIA team member to trigger CI again

[yechank-nvidia]

/bot run

[tensorrt-cicd]

PR_Github #34862 [ run ] triggered by Bot. Commit: f098163

[tensorrt-cicd]

PR_Github #34862 [ run ] completed with state SUCCESS. Commit: f098163
/LLM/release-1.2/L0_MergeRequest_PR pipeline #326 completed with status: 'SUCCESS'