Prhub

#38937 [ROCm][CI] Added back missing common deps

原始 PR 作者 AndreasKaratzas 合并时间 2026-04-04 06:58 文件变更 2 提交数 1 评论 14 代码增减 +284 / -10

执行摘要

恢复 ROCm CI 测试依赖中的通用依赖项,修复 AMD CI 环境。

根据PR body,作者承认在之前的PR中错误移除了通用依赖,需要添加回来以修复AMD CI测试环境。Issue评论中进一步讨论到-c和-r选项的区别以及compressed-tensors的兼容性风险。

对于负责CI或ROCm维护的工程师,建议关注此PR以理解依赖管理策略。变更较简单,但讨论揭示了-c与-r的微妙区别以及外部依赖兼容性的重要性。

讨论亮点

Review中,hmellor询问是否因此可以移除-c选项,引发了对依赖管理设计的讨论。Issue评论中深入探讨了-c与-r的区别,以及compressed-tensors pin到transformers<5可能导致AMD CI v5失败。结论是hmellor指出添加所有通用依赖可能破坏CI,建议更精细地添加缺失依赖,但PR已合并。

实现拆解

实现方案简单直接:在requirements/rocm-test.in文件中添加一行'-r common.txt',引用通用依赖文件。这导致pip-compile工具更新requirements/rocm-test.txt,添加了282行新依赖并删除10行,包括FastAPI组件、AI SDKs和其他工具库。

文件 模块 状态 重要度
requirements/rocm-test.in ci modified 5.0
requirements/rocm-test.txt ci modified 4.0

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

评论区精华

Dependency management with -c vs -r 设计

hmellor 询问是否可移除 -c 选项,讨论 -c 和 -r 在 pip-compile 中的作用区别。

结论:hmellor 指出 -c 可能多余,但 AndreasKaratzas 认为仍有作用,未达成明确结论。 · discussed

Compressed-tensors compatibility issue 正确性

hmellor 指出添加 common.txt 会引入 compressed-tensors,其 pin 到 transformers<5,可能破坏 AMD CI v5。

结论:建议需要更精细修复,而不是添加所有通用依赖;后续计划修复 compressed-tensors 项目。 · partially resolved

风险与影响

主要风险:引入compressed-tensors(版本0.14.0.1),其pin到transformers<5,可能与AMD CI v5中使用的transformers v5冲突,导致测试失败。此外,大量新依赖可能增加环境复杂性和潜在冲突。

影响范围:仅限于ROCm平台的CI测试环境。影响程度:中,修复了依赖缺失问题,但可能引入新兼容性问题,需要监控CI运行状态。

依赖冲突风险 测试环境不稳定

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论