[Cherry-Pick] update fleet_ops(#7859)#7858
Conversation
|
Thanks for your contribution! |
PaddlePaddle-bot
left a comment
PaddlePaddle-bot
left a comment
There was a problem hiding this comment.
🤖 Paddle-CI-Agent | pr_review |
2026-05-19 20:30:34
📋 Review 摘要
PR 概述：将 paddlefleet.ops.deep_ep 和 paddlefleet.ops.deep_gemm 的 import 路径更新为 paddlefleet_ops.deep_ep 和 paddlefleet_ops.deep_gemm，适配 paddlefleet 包重构后的新模块路径。
变更范围：model_executor/layers/moe/ep.py、model_executor/layers/quantization/fp8_utils.py
影响面 Tag：[OP] [Quantization]
问题
未发现阻塞性问题。
📝 PR 规范检查
PR 存在以下规范问题：
- 标题缺少 Tag 且不符合 Cherry-Pick 格式：目标分支为
release/2.6（非develop），标题须遵循[Cherry-Pick][Tag] 描述(#原PR号)格式，当前标题update fleet_ops既无 Tag 也未加[Cherry-Pick]前缀。 - 描述模板各段落均为空：
Motivation、Modifications、Usage or Command、Accuracy Tests均未填写，Checklist 均未勾选。
标题建议（可直接复制）：
[Cherry-Pick][BugFix] Fix paddlefleet_ops import path for deep_ep and deep_gemm(#原PR号)
PR 描述建议（可直接复制，必须复刻 checklist §D2 模板的完整结构）：
## Motivation
将 `paddlefleet.ops.deep_ep` 和 `paddlefleet.ops.deep_gemm` 的 import 路径更新为 `paddlefleet_ops.deep_ep` 和 `paddlefleet_ops.deep_gemm`，适配 paddlefleet 包重构后的新顶层包名。
## Modifications
- `fastdeploy/model_executor/layers/moe/ep.py`：将 `import paddlefleet.ops.deep_ep as deep_ep` 更改为 `import paddlefleet_ops.deep_ep as deep_ep`
- `fastdeploy/model_executor/layers/quantization/fp8_utils.py`：将 `import paddlefleet.ops.deep_gemm as deep_gemm` 更改为 `import paddlefleet_ops.deep_gemm as deep_gemm`
## Usage or Command
N/A
## Accuracy Tests
N/A
## Checklist
- [x] Add at least a tag in the PR title.
- Tag list: [`[FDConfig]`,`[APIServer]`,`[Engine]`, `[Scheduler]`, `[PD Disaggregation]`, `[Executor]`, `[Graph Optimization]`, `[Speculative Decoding]`, `[RL]`, `[Models]`, `[Quantization]`, `[Loader]`, `[OP]`, `[KVCache]`, `[DataProcessor]`, `[BugFix]`, `[Docs]`, `[CI]`, `[Optimization]`, `[Feature]`, `[Benchmark]`, `[Others]`, `[XPU]`, `[HPU]`, `[GCU]`, `[DCU]`, `[Iluvatar]`, `[Metax]`]
- You can add new tags based on the PR content, but the semantics must be clear.
- [x] Format your code, run `pre-commit` before commit.
- [x] Add unit tests. Please write the reason in this PR if no unit tests.（仅修改 import 路径，无逻辑变更，无需新增单测）
- [x] Provide accuracy results.（N/A，纯 import 路径变更，无精度影响）
- [ ] If the current PR is submitting to the `release` branch, make sure the PR has been submitted to the `develop` branch, then cherry-pick it to the `release` branch with the `[Cherry-Pick]` PR tag.总体评价
代码变更极小且清晰，仅修正 paddlefleet_ops 包的两处 import 路径，无逻辑错误。PR 规范方面需补全标题（添加 [Cherry-Pick][BugFix] 前缀及原 PR 号）和描述内容。
CI报告基于以下代码生成（30分钟更新一次）: 1 任务总览存在 1 个 Required 任务失败，需优先处理后方可合并。
2 任务状态汇总2.1 Required任务 : 9/10 通过
2.2 可选任务 — 22/26 通过
3 失败详情（仅 required）Run FastDeploy Unit Tests and Coverage / run_tests_with_coverage — 代码覆盖率不达标（置信度: 中）run_tests_with_coverage
失败步骤:
根因详情: 本 PR 仅修改了 2 个 import 路径（ 修复建议:
修复建议摘要: 在 import 行添加 |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## release/2.6 #7858 +/- ##
==============================================
Coverage ? 72.37%
==============================================
Files ? 381
Lines ? 54215
Branches ? 8473
==============================================
Hits ? 39237
Misses ? 12215
Partials ? 2763
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
3 participants
Motivation
将
paddlefleet.ops.deep_ep和paddlefleet.ops.deep_gemm的 import 路径更新为paddlefleet_ops.deep_ep和paddlefleet_ops.deep_gemm，适配 paddlefleet 包重构后的新顶层包名。Modifications
fastdeploy/model_executor/layers/moe/ep.py：将import paddlefleet.ops.deep_ep as deep_ep更改为import paddlefleet_ops.deep_ep as deep_epfastdeploy/model_executor/layers/quantization/fp8_utils.py：将import paddlefleet.ops.deep_gemm as deep_gemm更改为import paddlefleet_ops.deep_gemm as deep_gemmUsage or Command
N/A
Accuracy Tests
N/A
Checklist
[FDConfig],[APIServer],[Engine],[Scheduler],[PD Disaggregation],[Executor],[Graph Optimization],[Speculative Decoding],[RL],[Models],[Quantization],[Loader],[OP],[KVCache],[DataProcessor],[BugFix],[Docs],[CI],[Optimization],[Feature],[Benchmark],[Others],[XPU],[HPU],[GCU],[DCU],[Iluvatar],[Metax]]pre-commitbefore commit.releasebranch, make sure the PR has been submitted to thedevelopbranch, then cherry-pick it to thereleasebranch with the[Cherry-Pick]PR tag.