Prhub

#7403 [doc] chore: Change requirements install order

原始 PR 作者 MrJVium 合并时间 2026-08-14 11:35 文件变更 2 提交数 1 评论 0 代码增减 +2 / -2

执行摘要

调整 NPU 依赖安装顺序,避免被 setup.py 覆盖

PR body 明确指出目标是 "Avoid requirements-npu.txt overwrited by setup.py"。原安装顺序中,先装 requirements-npu.txt 再执行 pip install -e .setup.py 的依赖解析可能重装或降级刚装好的 NPU 包,导致环境异常;交换顺序后 requirements-npu.txt 的依赖在最后落地,不会被覆盖。

不值得精读,属于一次微小的安装流程修正。可作为了解 verl 对 Ascend/NPU 安装链路维护方式的参考:项目会同时维护脚本与文档的一致性,并关注 setup.py 对 NPU 依赖的干扰问题。若后续有更系统的依赖管理方案(如将 NPU 依赖纳入 extras),可再跟进。

讨论亮点

该 PR 没有任何 review 评论或讨论线程,唯一审核人 wucong25 直接通过(APPROVED)。变更内容简单明确,未引发设计或正确性争议。

实现拆解

  1. 修改 scripts/install_vllm_mcore_npu.sh:将 pip install -r requirements-npu.txt ...pip install -v -e . 两行顺序互换,使可编辑安装先完成、NPU 依赖后装,避免 setup.py 覆盖。
  2. 同步更新 docs/ascend_tutorial/get_start/install_guidance_A5.rst:文档中的两个安装指令同样互换顺序,与脚本保持一致,防止用户按文档操作时遭遇依赖覆盖问题。
  3. 无测试、配置或部署配套改动;本变更只影响安装步骤顺序,范围限定在 Ascend(NPU)安装链路。
文件 模块 状态 重要度
scripts/install_vllm_mcore_npu.sh 安装脚本 modified 2.35
docs/ascend_tutorial/get_start/install_guidance_A5.rst 安装文档 modified 1.32

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

评论区精华

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

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

风险与影响

风险很低。核心风险在于:pip install -e . 之后安装 requirements-npu.txt,如果其中存在与 setup.py 依赖冲突的版本,最终以 requirements-npu.txt 为准,这正是预期行为;但如果后续有人再次运行 pip install -e .pip install .,仍可能重新覆盖这些依赖。另外,scripts/install_vllm_mcore_npu.sh 与文档两处顺序现已一致,若未来只改其中一处可能造成流程漂移。

影响的是 Ascend(NPU)用户的安装体验:按脚本或 A5 文档安装 verl 时,NPU 特定依赖不会被 setup.py 覆盖,减少环境异常。影响范围限于安装流程,不涉及运行时逻辑;对团队而言是一次低成本的流程修正,能减少 NPU 环境的配置类 issue。

低风险 安装流程调整

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论