[https://nvbugs/6400067][test] Unwaive passing GB200 DeepSeek tests#16441
[https://nvbugs/6400067][test] Unwaive passing GB200 DeepSeek tests#16441litaotju wants to merge 2 commits into
Conversation
Signed-off-by: Tao Li @ NVIDIA <tali@nvidia.com>
📝 WalkthroughWalkthroughRemoved three previously waived integration test cases from Changes
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 |
|
/bot run |
|
PR_Github #59478 [ run ] triggered by Bot. Commit: |
|
PR_Github #59478 [ run ] completed with state
|
There was a problem hiding this comment.
Two of the three unwaived lines are under this PR's bug (nvbugs/6400067), but the third — full:GB200/test_e2e.py::test_qwen_e2e_cpprunner_large_new_tokens[DeepSeek-R1-Distill-Qwen-1.5B…] — is waived under a different bug, nvbugs/6414760, which the PR title/description never mention. Could you confirm 6414760 is actually resolved and either scope it out of this PR or call it out explicitly?
|
/bot run |
Summary by CodeRabbit
Description
Test Coverage
GitHub Bot Help
To see a list of available CI bot commands, please comment
/bot help.