-
Notifications
You must be signed in to change notification settings - Fork 1.8k
[TRTLLM-8638][fix] Add failed cases into waives.txt #8669
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[TRTLLM-8638][fix] Add failed cases into waives.txt #8669
Conversation
4cc356e to
2cb60f1
Compare
|
/bot run --skip-test |
📝 WalkthroughWalkthroughThis PR modifies test configuration files to manage test execution: removing a single test entry from the LLM function test list and adding SKIP entries for specific failing tests in the waives list. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes
Possibly related PRs
Suggested reviewers
Pre-merge checks and finishing touches❌ Failed checks (2 warnings)
✅ Passed checks (1 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
|
PR_Github #22553 [ run ] triggered by Bot. Commit: |
2cb60f1 to
f812f0a
Compare
Signed-off-by: xinhe-nv <[email protected]>
Signed-off-by: Xin He (SW-GPU) <[email protected]>
f812f0a to
5e6cbf2
Compare
|
PR_Github #22553 [ run ] completed with state |
|
/bot reuse-pipeline |
|
PR_Github #22585 [ reuse-pipeline ] triggered by Bot. Commit: |
|
PR_Github #22585 [ reuse-pipeline ] completed with state |
waive failed cases.
Summary by CodeRabbit