Prhub

#38714 Add ibm-granite/granite-vision-3.3-2b to supported models documentation

原始 PR 作者 jesus-talavera-ibm 合并时间 2026-04-01 23:22 文件变更 1 提交数 1 评论 3 代码增减 +1 / -1

执行摘要

在支持模型文档中添加 Granite Vision 模型条目。

根据 PR body,用户搜索文档中的“granite vision”时找不到任何信息,从而错误地得出结论认为该模型不受支持。实际上,Granite Vision 已通过 LlavaNextForConditionalGeneration 后端在 vLLM 中正常工作,并且已在 tests/models/registry.py 中注册,但完全缺失于 supported_models.md 文档。因此需要添加文档条目以提供准确的用户指引。

该 PR 变更简单直接,无需深入技术分析。对于维护文档完整性的团队成员值得快速浏览以确认模型支持信息的准确性。

讨论亮点

review 中几乎没有技术讨论。gemini-code-assist[bot] 评论指出这是文档更新,没有反馈。mgoin 直接批准。没有争议点或未解决疑虑。

实现拆解

仅修改了一个文件 docs/models/supported_models.md。在表格中 LlavaNextForConditionalGeneration 对应的行,将描述从“LLaVA-NeXT”更新为“LLaVA-NeXT, Granite Vision”,并在示例模型列表中添加了 ibm-granite/granite-vision-3.3-2b

文件 模块 状态 重要度
docs/models/supported_models.md 文档 modified 2.0

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

文档更新确认 documentation

gemini-code-assist[bot] 评论指出这是文档更新,没有反馈;mgoin 直接批准。

结论:变更被接受,无争议。 · 已解决

风险与影响

风险极低。这是纯文档变更,不涉及代码逻辑、性能、安全或兼容性。唯一潜在风险是文档准确性,但 PR 说明模型已通过现有后端支持且已在测试注册表中,因此风险可忽略。

影响范围仅限于文档用户。解决了用户因文档缺失而产生的困惑,提升了文档的完整性和准确性。对系统、团队或代码库无直接影响。

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

执行摘要

本 PR 在 docs/models/supported_models.md 文档中添加了 Granite Vision 模型条目,解决了用户因文档缺失而误认为该模型不受支持的问题。这是一个纯文档变更,风险极低,仅影响文档准确性,对代码库无直接影响。

功能与动机

根据 PR body,用户搜索文档中的“granite vision”时找不到任何信息,从而错误地得出结论认为该模型不受支持。实际上,Granite Vision 已通过 LlavaNextForConditionalGeneration 后端在 vLLM 中正常工作,并且已在 tests/models/registry.py 中注册,但完全缺失于 supported_models.md 文档。因此需要添加文档条目以提供准确的用户指引。

实现拆解

仅修改了一个文件 docs/models/supported_models.md。具体变更如下:

文件 变更内容
docs/models/supported_models.md LlavaNextForConditionalGeneration 行中,将描述从“LLaVA-NeXT”更新为“LLaVA-NeXT, Granite Vision”,并在示例模型列表中添加 ibm-granite/granite-vision-3.3-2b

变更前后对比(基于 patch_excerpt):

-| `LlavaNextForConditionalGeneration` | LLaVA-NeXT | T + I<sup>E+</sup> | `llava-hf/llava-v1.6-mistral-7b-hf`, `llava-hf/llava-v1.6-vicuna-7b-hf`, etc. | | ✅︎ |
+| `LlavaNextForConditionalGeneration` | LLaVA-NeXT, Granite Vision | T + I<sup>E+</sup> | `llava-hf/llava-v1.6-mistral-7b-hf`, `llava-hf/llava-v1.6-vicuna-7b-hf`, `ibm-granite/granite-vision-3.3-2b`, etc. | | ✅︎ |

评论区精华

review 中几乎没有技术讨论:

  • gemini-code-assist[bot] 评论:“This pull request updates the documentation for supported models by adding Granite Vision to the LlvaNextForConditionalGeneration entry and including ibm-granite/granite-vision-3.3-2b as an example. I have no feedback to provide.”
  • mgoin 直接批准,无额外评论。

没有争议点或未解决疑虑,变更被顺利接受。

风险与影响

风险分析:风险极低。这是纯文档变更,不涉及代码逻辑、性能、安全或兼容性。唯一潜在风险是文档准确性,但 PR 说明模型已通过现有后端支持且已在测试注册表中,因此风险可忽略。

影响分析:影响范围仅限于文档用户。解决了用户因文档缺失而产生的困惑,提升了文档的完整性和准确性。对系统、团队或代码库无直接影响。

关联脉络

与近期历史 PR 的关联:

  • PR #38722 和 #38723 同为文档类小修复,涉及 documentation 和 cleanup 标签,体现了团队对文档细节的持续维护。
  • 本 PR 延续了多模态模型(如 Granite Vision)文档支持的完善趋势,与仓库中 multi-modality 标签的 PR 相呼应。

整体来看,这是一个简单的文档补充,旨在确保用户能准确获取模型支持信息,避免因文档缺失导致的误解。

参与讨论