Skip to content

Update submodule DeepEP#762

Draft
youge325 wants to merge 1 commit intoPaddlePaddle:developfrom
youge325:cDeepEP
Draft

Update submodule DeepEP#762
youge325 wants to merge 1 commit intoPaddlePaddle:developfrom
youge325:cDeepEP

Conversation

@youge325
Copy link
Copy Markdown
Contributor

将 DeepEP 更新到最新 commit ,前置 pr :PFCCLab/DeepEP#11

Copilot AI review requested due to automatic review settings April 12, 2026 08:56
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

该 PR 将项目依赖的 DeepEP Git submodule 更新到上游最新 commit(对应前置 PR https://github.com/PFCCLab/DeepEP/pull/11),以便在本仓库的构建/自定义算子生态中使用更新后的 DeepEP 版本。

Changes:

  • 更新 third_party/DeepEP 子模块指针到最新 commit

@youge325
Copy link
Copy Markdown
Contributor Author

/re-run all-failed

@youge325
Copy link
Copy Markdown
Contributor Author

https://www.paddlepaddle.org.cn/packages/nightly/cu130/paddlepaddle-gpu/ 当中有更新的 3.3.1.post 包再改 build_utils.py: 227,还是直接改 4.0.0dev 包

@SigureMo
Copy link
Copy Markdown
Member

https://www.paddlepaddle.org.cn/packages/nightly/cu130/paddlepaddle-gpu/ 当中有更新的 3.3.1.post 包再改 build_utils.py: 227,还是直接改 4.0.0dev 包

这边依赖的 post 包是 release/3.3 产出的,需要 develop 分支 cherry-pick 到 release/3.3 分支才会产出这些

这里有两个选择:

  1. 将相关改动 cherry-pick 到 release/3.3,但就我目前的观察,C++ 接口改动非常多,cherry-pick 成本很高
  2. 等月底 3.4 拉分支后,PaddleFleet 依赖 Paddle 3.4 后再更新

另外,我目前跟进的 HybridEP 后续有望替代掉 DeepEP,HybridEP 分支比 DeepEP 与上游 diff 更大,因为目前仍然有基于 release/3.3 使用的需求,用不了 develop 新增的一些接口,也就是说现在在 DeepEP 上的工作可能在 HybridEP 上再做一遍,因此我建议不如在整体切到 3.4 之后再做(直接基于 HybridEP 分支做),现在我们主要验证我们在 develop 上开发的接口确实有降低迁移成本、减少 diff 的能力即可

PaddlePaddle/Paddle#78549 建议也暂不合入吧,我们保证在 3.4 拉分支前是同时包含新老接口的,这样留一个版本的 buffer,3.4 拉分支后再清理掉

相关 PR 可以暂时转 draft,待时机合适再推进

@youge325 youge325 marked this pull request as draft April 12, 2026 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants