[None][infra] Waive 1 failed cases for main in pre-merge 54650 - #17925
Conversation
Bug(s): 6581049 Requested by: @pranav-nvidia Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review. WalkthroughThe integration waiver list skips the NanoV3 Omni ChangesNanoV3 Omni test waiver
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This change only waives one specific failing test case tied to the referenced bug. No actionable merge-blocking risk remains after normal checks and review. Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
/bot skip --comment "Waiving broken test" |
|
PR_Github #67150 [ skip ] triggered by Bot. Commit: |
|
PR_Github #67150 [ skip ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by @pranav-nvidia)
Target branch:
mainBug(s): 6581049
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Dev Engineer Review
accuracy/test_llm_api_pytorch_multimodal.py::TestNanoV3Omni::test_auto_dtype[fp8_mmmu_encoder_cuda_graph].6581049and limits the waiver to the intended failed test.QA Engineer Review
test-db/orqa/files were modified.tests/integration/test_lists/waives.txt.