Repository navigation
[None][infra] Waive 1 failed cases for main in pre-merge 48383 - #16560
trtllm-agent wants to merge 1 commit into
Conversation
Bug(s): 6367792 Requested by: @longlee0622 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)
📝 WalkthroughWalkthroughAdded a SKIP waiver for the Nemotron Nano V3 accuracy test’s ChangesIntegration test waiver
Estimated code review effort: 1 (Trivial) | ~2 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 |
BowenFu
left a comment
There was a problem hiding this comment.
LGTM — a single scoped CI waiver for TestNemotronNanoV3::test_accuracy[fp8-4-attn_dp_off-trtllm] with a valid nvbug link (6367792); no product code.
|
Bug engineer ownership @tcherckez-nvidia, as the engineer assigned to the linked bug, you are responsible for driving this bot PR to resolution. Please coordinate review, address feedback and required code changes, get CI passing, and verify the original failure before driving it to merge. If the PR is obsolete or superseded, close it with a reason. Keep this PR updated with blockers and the next step. |
|
not relevant. test is not in waives |
Auto-generated Waive PR
Created by: TensorRT LLM CI Report (requested by @longlee0622)
Target branch:
mainBug(s): 6367792
Waive entries added
This PR was auto-generated by TensorRT LLM CI Report. Please review the waive entries before merging.
Summary by CodeRabbit