- Notifications
You must be signed in to change notification settings - Fork1.9k
[None][fix] Update DG commit#7534
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
[None][fix] Update DG commit#7534
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Signed-off-by: Barry Kang <43644113+Barry-Delaney@users.noreply.github.com>
Barry-Delaney commentedSep 4, 2025
/bot run |
📝 WalkthroughWalkthroughUpdated the 3rdparty/DeepGEMM submodule reference from commit 7b6b5563b9d4c1ae07ffbce7f78ad3ac9204827c to 67e3c4d3d09b59405fd6e7698a33db747ed96533. No other changes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat withCodeRabbit:
SupportNeed help? Create a ticket on oursupport page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
Status, Documentation and Community
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Actionable comments posted: 1
🧹 Nitpick comments (1)
3rdparty/DeepGEMM (1)
1-1:Optional: prefer a tagged ref for release branches.If an upstream tag covers this commit, consider pinning to the tag in release branches to improve traceability.
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
3rdparty/DeepGEMM(1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
**/*
📄 CodeRabbit inference engine (CODING_GUIDELINES.md)
Filenames compiled into a target must be case-insensitively unique
Files:
3rdparty/DeepGEMM
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Pre-commit Check
🔇 Additional comments (2)
3rdparty/DeepGEMM (2)
1-1:LGTM — submodule pointer corrected.Change is scoped to updating the DeepGEMM submodule commit. No issues spotted.
1-1:Manually verify DeepGEMM LICENSE/NOTICE changes
Previous submodule commit couldn’t be auto-determined, so please ensure the updated revision’s LICENSE, NOTICE, and README files haven’t changed licensing or attribution without updating our third-party docs.
Uh oh!
There was an error while loading.Please reload this page.
tensorrt-cicd commentedSep 4, 2025
PR_Github #17662 [ run ] triggered by Bot |
tensorrt-cicd commentedSep 4, 2025
PR_Github #17662 [ run ] completed with state |
9644d24 intoNVIDIA:release/1.1.0rc2Uh oh!
There was an error while loading.Please reload this page.
Signed-off-by: Barry Kang <43644113+Barry-Delaney@users.noreply.github.com>
Signed-off-by: Barry Kang <43644113+Barry-Delaney@users.noreply.github.com>
Signed-off-by: Barry Kang <43644113+Barry-Delaney@users.noreply.github.com>
Signed-off-by: Barry Kang <43644113+Barry-Delaney@users.noreply.github.com>
Signed-off-by: Barry Kang <43644113+Barry-Delaney@users.noreply.github.com>
Uh oh!
There was an error while loading.Please reload this page.
This PR corrects the commit ID in
3rdparty/DeepGEMM.Summary by CodeRabbit