-
Notifications
You must be signed in to change notification settings - Fork 2k
[Doc][Qwen3] update qwen3 into support-matrix #6161
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
Conversation
Signed-off-by: bhsueh <[email protected]>
WalkthroughThe documentation was updated to add support for two new Qwen3-based models, specifying their HuggingFace examples and modality tags. The reference link for Qwen models in the TensorRT backend section was also expanded to include Qwen3. Changes
Poem
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
|
/bot skip --comment "only update doc" |
|
/bot skip --comment "only update doc" |
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: 0
🧹 Nitpick comments (1)
docs/source/reference/support-matrix.md (1)
28-29: Align MoE naming and double-check the HF example slug
Qwen3MoeForCausalLMvs.Qwen3MoEmixes the “e/E” capitalisation. Pick one convention (the repo usually uses MoE) for both the architecture column and the friendly model name to avoid search / copy-paste friction.
While touching the row, please confirm the HF repo is the exact slug (Qwen/Qwen3-30B-A3Blooks uncommon; many repos use…-A32Bor…-MixMoE).
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/source/reference/support-matrix.md(2 hunks)
🔇 Additional comments (1)
docs/source/reference/support-matrix.md (1)
77-77: LGTM – link update accurately reflects Qwen3 addition
No issues found with the modified bullet.
|
PR_Github #12267 [ skip ] triggered by Bot |
|
PR_Github #12267 [ skip ] completed with state |
Signed-off-by: bhsueh <[email protected]>
Signed-off-by: bhsueh <[email protected]>
Signed-off-by: bhsueh <[email protected]>
Signed-off-by: bhsueh <[email protected]>
Signed-off-by: bhsueh <[email protected]>
Signed-off-by: bhsueh <[email protected]> Signed-off-by: Shreyas Misra <[email protected]>
Signed-off-by: bhsueh <[email protected]> Signed-off-by: Ransiki Zhang <[email protected]>
Summary by CodeRabbit