Prhub

#47072 [CI][Bugfix] Add cohere_melody to ROCm test requirements

原始 PR 作者 peizhang56 合并时间 2026-06-30 05:29 文件变更 2 提交数 1 评论 0 代码增减 +3 / -0

执行摘要

修复 ROCm CI 缺少 cohere_melody 依赖

ROCm CI 运行 pytest -v -s reasoning 时,tests/reasoning/test_cohere_command_reasoning_parser.py 硬导入 cohere_melody 包,但该包仅声明在 requirements/test/cuda.in 中,导致 ROCm 镜像缺少该依赖,测试收集阶段失败。

建议合并无风险变更。可提醒团队将新依赖同步添加至其他平台(如 Intel GPU)的 requirements.in 文件,若其也运行相同测试。

讨论亮点

无 review 讨论。PR 由 Claude 辅助生成,经提交者测试验证,由 AndreasKaratzas 批准。

实现拆解

  1. requirements/test/rocm.in 第 77 行添加 cohere_melody>=0.9.0 依赖,并注明用于 cohere command reasoning parser 测试。
  2. 重新生成 requirements/test/rocm.txt 锁定文件,新增 cohere-melody==0.9.0 条目及其 via 注释。
  3. 该包为纯 CPU wheel,无 CUDA/ROCm 链接或传递依赖,因此无需额外适配。
文件 模块 状态 重要度
requirements/test/rocm.in 测试依赖 modified 3.97
requirements/test/rocm.txt 锁定文件 modified 1.89

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

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

风险极低:仅添加一个纯 Python 依赖包,无二进制编译,不影响已有功能。

正面影响:修复 ROCm CI 测试收集失败,使 cohere command reasoning 测试套件(40 个测试用例)能在 ROCm 平台上正常执行。

低风险依赖变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论