执行摘要
- 一句话:修复 ROCm CI 缺少 cohere_melody 依赖
- 推荐动作:建议合并无风险变更。可提醒团队将新依赖同步添加至其他平台(如 Intel GPU)的 requirements.in 文件,若其也运行相同测试。
功能与动机
ROCm CI 运行 pytest -v -s reasoning 时,tests/reasoning/test_cohere_command_reasoning_parser.py 硬导入 cohere_melody 包,但该包仅声明在 requirements/test/cuda.in 中,导致 ROCm 镜像缺少该依赖,测试收集阶段失败。
实现拆解
- 在
requirements/test/rocm.in 第 77 行添加 cohere_melody>=0.9.0 依赖,并注明用于 cohere command reasoning parser 测试。
- 重新生成
requirements/test/rocm.txt 锁定文件,新增 cohere-melody==0.9.0 条目及其 via 注释。
- 该包为纯 CPU wheel,无 CUDA/ROCm 链接或传递依赖,因此无需额外适配。
关键文件:
requirements/test/rocm.in(模块 测试依赖;类别 test;类型 test-coverage): 核心变更文件:添加缺失的 cohere_melody 依赖声明,修复 ROCm CI 测试收集错误
requirements/test/rocm.txt(模块 锁定文件;类别 docs;类型 documentation): 锁定文件同步更新,包含 cohere-melody 0.9.0 及其传递依赖
关键符号:未识别
评论区精华
无 review 讨论。PR 由 Claude 辅助生成,经提交者测试验证,由 AndreasKaratzas 批准。
风险与影响
- 风险:风险极低:仅添加一个纯 Python 依赖包,无二进制编译,不影响已有功能。
- 影响:正面影响:修复 ROCm CI 测试收集失败,使 cohere command reasoning 测试套件(40 个测试用例)能在 ROCm 平台上正常执行。
- 风险标记:低风险依赖变更
关联脉络
参与讨论