执行摘要
- 一句话:修复 Kimi K3 子包缺失 init.py 导致的问题
- 推荐动作:可直接合并。建议日后在新增子包时使用
python -m build 验证 wheel 完整性。
功能与动机
PR #50090 新增了 vllm/models/kimi_k3/ 目录,但未添加 __init__.py,使其成为隐式命名空间包。这导致文档构建(mkdocstrings/Griffe)无法解析子模块,并且 setuptools 的 packages.find 无法识别该包,构建的 wheel 中缺少 vllm.models.kimi_k3 及其所有子模块。
实现拆解
- 在
vllm/models/kimi_k3/ 下新建 __init__.py,仅包含 Apache-2.0 许可的 SPDX 头信息,使该目录成为显式 Python 包。
- 修改
vllm/models/kimi_k3/amd/ops/__init__.py,为原本为空的文件添加相同的 SPDX 头。
- 该变更仅涉及打包基础设施,不影响运行时模型代码。
关键文件:
vllm/models/kimi_k3/__init__.py(模块 模型注册;类别 source;类型 data-contract): 新增文件,使 kimi_k3 成为显式包,直接修复文档构建和 wheel 打包问题。
vllm/models/kimi_k3/amd/ops/__init__.py(模块 模型注册;类别 infra;类型 infrastructure): 补充 SPDX 许可头,保持仓库风格一致。
关键符号:未识别
评论区精华
无 review 讨论。该 PR 由两名维护者审批通过,无未解决疑虑。
风险与影响
关联脉络
- PR #50089 [Model] Add Kimi K3 support: model files and kernels [1/N]: 该 PR 引入了 kimi_k3 目录,但未添加 init.py,直接导致本修复 PR 的必要性。
- PR #50093 [Model] Add Kimi K3 support: Python frontend [2/2]: 同属 Kimi K3 支持系列 PR,本修复 PR 为前者提供正确的包结构前提。
参与讨论