执行摘要
- 一句话:移除 TeleChatForCausalLM 模型别名
- 推荐动作:该 PR 值得快速合并,属于低风险、高收益的清理变更。值得关注的设计决策是将弃用的模型以真实版本号添加到
_PREVIOUSLY_SUPPORTED_MODELS 中,这为未来进一步的清理工作(如 0.26.0 后完全移除相关代码)奠定了基础。
功能与动机
低使用量,近 3 个月使用时间不足 10 小时。PR body 明确指出:"Remove TeleChatForCausalLM due to low usage (less than 10 hours in the past 3 months)"。这是标准的代码清理策略,通过废弃低采纳率的遗留兼容别名来减小维护负担。
实现拆解
- 移除注册表中的别名:在
vllm/model_executor/models/registry.py 中,从 _TEXT_GENERATION_MODELS 和 _EMBEDDING_MODELS 两个字典中删除了 "TeleChatForCausalLM" 条目,这两个条目原来都映射到 ("telechat2", "TeleChat2ForCausalLM")。
- 标记废弃版本:在同一个文件的
_PREVIOUSLY_SUPPORTED_MODELS 字典中新增 "TeleChatForCausalLM": "0.25.0",明确声明该模型在 0.25.0 版本之后不再支持(经过 review 讨论后调整为 0.25.0)。
- 更新测试数据:在
tests/models/registry.py 的 _TEXT_GENERATION_EXAMPLE_MODELS 字典中删除 TeleChatForCausalLM 的测试条目,该条目引用了 chuhac/TeleChat2-35B 模型检查点。
- 同步文档:在
docs/models/supported_models.md 的支持表格中删除 TeleChatForCausalLM 行,避免给用户造成该别名仍受支持的误导。
关键文件:
vllm/model_executor/models/registry.py(模块 模型注册表;类别 source;类型 data-contract): 核心变更文件。从 _TEXT_GENERATION_MODELS 和 _EMBEDDING_MODELS 中删除别名,并在 _PREVIOUSLY_SUPPORTED_MODELS 中记录废弃版本。
tests/models/registry.py(模块 测试;类别 test;类型 test-coverage): 配套测试文件。删除与废弃别名对应的测试用例(引用 chuhac/TeleChat2-35B 检查点)。
docs/models/supported_models.md(模块 文档;类别 docs;类型 documentation): 文档同步更新。删除支持表格中 TeleChatForCausalLM 的行,避免误导用户。
关键符号:未识别
关键源码片段
vllm/model_executor/models/registry.py
核心变更文件。从 _TEXT_GENERATION_MODELS 和 _EMBEDDING_MODELS 中删除别名,并在 _PREVIOUSLY_SUPPORTED_MODELS 中记录废弃版本。
# vllm/model_executor/models/registry.py
# 在 _TEXT_GENERATION_MODELS 和 _EMBEDDING_MODELS 中删除以下行:
# "TeleChatForCausalLM": ("telechat2", "TeleChat2ForCausalLM"),
# 并在 _PREVIOUSLY_SUPPORTED_MODELS 中添加一行以记录最终支持版本:
# 注意:"0.25.0" 是最后一个包含该别名的版本
_PREVIOUSLY_SUPPORTED_MODELS = {
# ... 其他条目
"TeleChatForCausalLM": "0.25.0", # 在 0.26.0 版本中移除
}
评论区精华
在 review 中,对 _PREVIOUSLY_SUPPORTED_MODELS 中的版本号存在讨论:
风险与影响
关联脉络
参与讨论