[None][test] Waive 7 failed cases for main in QA CI - #16982
Conversation
Bug(s): 5929339, 6525059, 6525892, 6525893, 6525896, 6525897, 6525898 Requested by: qa@nvidia.com Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
|
/bot skip --comment "Only waive tests" |
WalkthroughThe GB200 waiver list replaces several existing accuracy skips with updated accuracy and multimodal test waivers across Llama, DeepSeek, MiniMax, Nemotron, Qwen, and Gemma configurations. ChangesGB200 waiver updates
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@tests/integration/test_lists/waives.txt`:
- Around line 186-204: Add the repository-standard NVIDIA copyright header at
the beginning of the modified waives.txt file, using the current required
copyright year for modified files; leave the existing waiver entries unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: fe0a00ae-ca29-4ae7-af08-c667bf19e5bb
📒 Files selected for processing (1)
tests/integration/test_lists/waives.txt
|
PR_Github #62375 [ skip ] triggered by Bot. Commit: |
|
PR_Github #62375 [ skip ] completed with state |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by qa@nvidia.com)
Target branch:
mainBug(s): 5929339, 6525059, 6525892, 6525893, 6525896, 6525897, 6525898
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Dev Engineer Review
tests/integration/test_lists/waives.txtwith seven GB200 accuracy-test waivers, each markedSKIPand associated with bugs5929339,6525059,6525892,6525893,6525896,6525897, and6525898.QA Engineer Review
tests/integration/test_lists/waives.txtwas modified.accuracyandaccuracy_multimodalcases covering Llama, DeepSeek, MiniMax, Nemotron, Qwen, and Gemma tests.test-db//qa/files were changed.