chore: update TRTLLM-Gen GEMM cubins - #4840
Conversation
Signed-off-by: Bo Li <22713281+bobboli@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe TRTLLM generated GEMM artifact path pin and its corresponding SHA256 checksum were updated in ChangesTRTLLM GEMM artifact
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This PR updates the pinned TRTLLM-Gen GEMM package and matching manifest checksum without changing retrieval or integrity-checking behavior. The change is localized and merge-ready after normal checks and review, with no actionable merge-blocking risk remaining. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Description checkExplanation The description follows the repository template and documents the artifact pin, checksum, validation steps, test status, and reviewer notes. It clearly states that the full test suite and all-file pre-commit run were not performed. Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files. ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 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 |
|
/bot run tests/gemm |
|
[FAILED] Pipeline #65443666 — 9/16 executed test jobs passed Compared with nightly #65302610. Unit Tests
✅ Pass · 🟡 Old failure · ❌ New failure · ⏱ Test timeout · Multi-GPU and Multi-Node Tests — 6/6 passed
Failure detailsNew relative to nightly (attribution uncertain)
Timeouts, infrastructure, or incomplete jobs
|
|
Follow-up on the reported The PR head was based on an older I re-ran the full test file against the actual merge commit, using the updated GEMM artifact ( No additional fix or cubin rollback is needed for this issue. |
📌 Description
Update the TRTLLM-Gen dense GEMM artifact pin to the newly published multi-architecture package:
7b1fc253cd6237950e76310873f4acf4d97a3904/gemm-b738138-25754e6/ca9d4f956f3fb63bff3066db88fa7ccf08b00f4b0b2751cc14ba72454fd0163893 sm100f + 93 sm107a)This brings in the TRTLLM-Gen dense GEMM fixes from revision
b738138while retaining the existing GEMM config hash25754e6.🔍 Related Issues
N/A
🚀 Pull Request Checklist
✅ Pre-commit Checks
pre-commit run --all-fileswas not run for this pin-only change.🧪 Tests
checksums.txtSHA-256 matches the new pin and contains 201 entries.Reviewer Notes
The artifact was published from cubin-publishing main pipeline
65421116, job418937481; public publish job418937489completed successfully.Summary by CodeRabbit