Skip to content

[None][infra] Waive 10 failed cases for main in post-merge 2636#12767

Merged
EmmaQiaoCh merged 1 commit intoNVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260405-142209
Apr 5, 2026
Merged

[None][infra] Waive 10 failed cases for main in post-merge 2636#12767
EmmaQiaoCh merged 1 commit intoNVIDIA:mainfrom
ZhanruiSunCh:trtllm-ci-report/waive-20260405-142209

Conversation

@ZhanruiSunCh
Copy link
Copy Markdown
Collaborator

@ZhanruiSunCh ZhanruiSunCh commented Apr 5, 2026

Auto-generated Waive PR

Created by: TensorRT LLM CI Report (requested by @EmmaQiaoCh)
Target branch: main
Bug(s): 6050481, 6050483, 6050487, 6050489

Waive entries added

accuracy/test_llm_api_pytorch.py::TestGPTOSS::test_w4_4gpus[v2_kv_cache-ep4-trtllm-fp8] SKIP (https://nvbugs/6050481)
examples/test_visual_gen.py::test_vbench_dimension_score_wan22_a14b_nvfp4 SKIP (https://nvbugs/6050483)
visual_gen/test_visual_gen_benchmark.py::test_online_benchmark[openai-videos] SKIP (https://nvbugs/6050483)
accuracy/test_llm_api_pytorch.py::TestGPTOSS::test_w4_4gpus[v1_kv_cache-ep4-trtllm-fp8] SKIP (https://nvbugs/6050481)
examples/test_visual_gen.py::test_vbench_dimension_score_wan SKIP (https://nvbugs/6050483)
examples/test_visual_gen.py::test_vbench_dimension_score_wan22_a14b_fp8 SKIP (https://nvbugs/6050483)
visual_gen/test_visual_gen_benchmark.py::test_offline_benchmark SKIP (https://nvbugs/6050483)
accuracy/test_llm_api_pytorch.py::TestLlama3_1_8BInstruct::test_bfloat16_4gpus[pp4-attn_backend=TRTLLM-torch_compile=False] SKIP (https://nvbugs/6050487)
accuracy/test_llm_api_pytorch.py::TestDeepSeekV3Lite::test_bfloat16_4gpus[pp4-mtp_nextn=0-attention_dp=False-cuda_graph=True-overlap_scheduler=False-torch_compile=False] SKIP (https://nvbugs/6050489)
accuracy/test_llm_api_pytorch.py::TestDeepSeekV3Lite::test_bfloat16_4gpus[pp4-mtp_nextn=0-attention_dp=False-cuda_graph=False-overlap_scheduler=False-torch_compile=False] SKIP (https://nvbugs/6050489)

This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.

Summary by CodeRabbit

  • Tests
    • Updated test skip list for pending validation of specific test configurations.

Bug(s): 6050481, 6050483, 6050487, 6050489
Requested by: @EmmaQiaoCh

Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
@EmmaQiaoCh
Copy link
Copy Markdown
Collaborator

/bot skip --comment "Waive failed cases"

@EmmaQiaoCh EmmaQiaoCh enabled auto-merge (squash) April 5, 2026 14:22
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 5, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: be2f7e23-2683-430f-8544-33b59c12488b

📥 Commits

Reviewing files that changed from the base of the PR and between 56e6961 and 8d414b6.

📒 Files selected for processing (1)
  • tests/integration/test_lists/waives.txt

📝 Walkthrough

Walkthrough

This PR adds 10 SKIP entries to the test waive list file, disabling specific failing test cases across accuracy/test_llm_api_pytorch.py, examples/test_visual_gen.py, and visual_gen/test_visual_gen_benchmark.py test modules without modifying existing entries.

Changes

Cohort / File(s) Summary
Test Waive List
tests/integration/test_lists/waives.txt
Added 10 SKIP entries for failing tests: TestGPTOSS::test_w4_4gpus (2 variants), test_vbench_dimension_score_wan* (3 variants), test_online_benchmark and test_offline_benchmark, TestLlama3_1_8BInstruct::test_bfloat16_4gpus, and TestDeepSeekV3Lite::test_bfloat16_4gpus (2 parameterizations).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • chzblych
  • EmmaQiaoCh
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive Description provides clear waive entries with NVBug references, though it lacks explicit sections matching the template (Description, Test Coverage, Checklist). Restructure description to explicitly include Description and Test Coverage sections as required by template, and ensure all checklist items are addressed.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed Title clearly identifies this as an infra change for waiving test cases and specifies the scope with PR identifier.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #41879 [ skip ] triggered by Bot. Commit: 8d414b6 Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #41879 [ skip ] completed with state SUCCESS. Commit: 8d414b6
Skipping testing for commit 8d414b6

Link to invocation

@EmmaQiaoCh EmmaQiaoCh merged commit 5548d01 into NVIDIA:main Apr 5, 2026
8 checks passed
yufeiwu-nv pushed a commit to yufeiwu-nv/TensorRT-LLM that referenced this pull request Apr 7, 2026
…IA#12767)

Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
karen-sy pushed a commit to karen-sy/TensorRT-LLM that referenced this pull request Apr 7, 2026
…IA#12767)

Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
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.

3 participants