[None][doc] Update Qwen2/3-VL's model on supported_models.md#10797
[None][doc] Update Qwen2/3-VL's model on supported_models.md#10797yechank-nvidia merged 2 commits intoNVIDIA:mainfrom
Conversation
Signed-off-by: yechank <161688079+yechank-nvidia@users.noreply.github.com>
📝 WalkthroughWalkthroughThis pull request updates the supported models documentation to reflect feature support for Qwen models. It enables a specific feature for two existing Qwen2 variants and adds two new Qwen3 model entries to the support matrix. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@docs/source/models/supported-models.md`:
- Around line 64-65: The table entry for Qwen2VLForConditionalGeneration
incorrectly claims EPD Disaggregated Serving support; update the table row for
`Qwen2VLForConditionalGeneration` to mark the EPD Disaggregated Serving column
as "No" (instead of "Yes"). Use the model's implementation details (the raised
NotImplementedError "Qwen2VLModel does not support disaggregated inference yet"
and the absence of the `@support_multimodal_disaggregated` decorator on
`Qwen2VLForConditionalGeneration`) as the justification for this change so the
docs match the code.
Signed-off-by: yechank <161688079+yechank-nvidia@users.noreply.github.com>
|
/bot skip --comment "doc changes" |
|
PR_Github #34190 [ skip ] triggered by Bot. Commit: |
|
PR_Github #34190 [ skip ] completed with state |
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.