Skip to content

[https://nvbugs/6401925][fix] Unwaive DeepSeek V4 compressor corner-case test#15957

Merged
yuxianq merged 1 commit into
NVIDIA:mainfrom
yuxianq:fix/nvbug-6401925-unwaive-compressor-corner-case
Jul 6, 2026
Merged

[https://nvbugs/6401925][fix] Unwaive DeepSeek V4 compressor corner-case test#15957
yuxianq merged 1 commit into
NVIDIA:mainfrom
yuxianq:fix/nvbug-6401925-unwaive-compressor-corner-case

Conversation

@yuxianq

@yuxianq yuxianq commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Tests
    • Removed one test skip entry, allowing that previously waived integration test case to run again.

Description

Remove the DGX B200 waiver for
test_prefill_corner_cases[overlap_large_head_dim] from waives.txt.

The test passed 100 consecutive fresh-process runs on a DGX B200 after a clean
B200 build, so the failure tracked by https://nvbugs/6401925 was not reproduced.

Test Coverage

  • Waiver parser: parsed all 626 remaining entries successfully.
  • python scripts/check_test_list.py --validate: 2371 unique entries validated.
  • DGX B200 at commit 0e9c8fb7cb0467bfcc0284d1f5354cc6115baecd:
    exact test selector passed 100/100 consecutive runs.

PR Checklist

Please review the following before submitting your PR:

  • PR description clearly explains what and why. If using CodeRabbit's summary, please make sure it makes sense.

  • PR Follows TRT-LLM CODING GUIDELINES to the best of your knowledge.

  • Test cases are provided for new code paths (see test instructions)

  • If PR introduces API changes, an appropriate PR label is added - either api-compatible or api-breaking. For api-breaking, include BREAKING in the PR title.

  • Any new dependencies have been scanned for license and vulnerabilities

  • CODEOWNERS updated if ownership changes

  • Documentation updated as needed

  • Update tava architecture diagram if there is a significant design change in PR.

  • The reviewers assigned automatically/manually are appropriate for the PR.

  • Please check this after reviewing the above items as appropriate for this PR.

GitHub Bot Help

To see a list of available CI bot commands, please comment /bot help.

Signed-off-by: Yuxian Qiu <142763828+yuxianq@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: c858413f-06da-47ea-b3a4-b45a16a100de

📥 Commits

Reviewing files that changed from the base of the PR and between 6c22551 and d6c3ef1.

📒 Files selected for processing (1)
  • tests/integration/test_lists/waives.txt
💤 Files with no reviewable changes (1)
  • tests/integration/test_lists/waives.txt

📝 Walkthrough

Walkthrough

A single waive entry for the DGX_B200 test test_prefill_corner_cases[overlap_large_head_dim] was removed from tests/integration/test_lists/waives.txt, so that test case is no longer marked to be skipped.

Changes

Waive list update

Layer / File(s) Summary
Remove skip entry
tests/integration/test_lists/waives.txt
Deletes the waive line for the DGX_B200 test_prefill_corner_cases[overlap_large_head_dim] test case, re-enabling that test.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: hyukn

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title follows the required ticket-and-type format and clearly summarizes the waiver removal.
Description check ✅ Passed The description matches the template with summary, description, test coverage, and checklist sections filled in.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@yuxianq

yuxianq commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@yuxianq yuxianq requested a review from mzweilz July 6, 2026 03:30
@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #57679 [ run ] triggered by Bot. Commit: d6c3ef1 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #57679 [ run ] completed with state SUCCESS. Commit: d6c3ef1
/LLM/main/L0_MergeRequest_PR pipeline #46399 completed with status: 'SUCCESS'

CI Report

Link to invocation

@yuxianq yuxianq merged commit ca708a1 into NVIDIA:main Jul 6, 2026
11 checks passed
BrianLi23 pushed a commit to BrianLi23/TensorRT-LLM that referenced this pull request Jul 9, 2026
…ase test (NVIDIA#15957)

Signed-off-by: Yuxian Qiu <142763828+yuxianq@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants