update fleet_ops#7859
Conversation
|
Thanks for your contribution! |
PaddlePaddle-bot
left a comment
There was a problem hiding this comment.
🤖 Paddle-CI-Agent | pr_review |
2026-05-19 20:30:35
📋 Review 摘要
PR 概述:将 MoE EP 及 FP8 量化模块中 PaddleFleet 算子的导入路径从 paddlefleet.ops 更新为 paddlefleet_ops,适配上游包重命名。
变更范围:fastdeploy/model_executor/layers/moe/ep.py、fastdeploy/model_executor/layers/quantization/fp8_utils.py
影响面 Tag:[OP] [Quantization]
问题
未发现阻塞性问题。
📝 PR 规范检查
PR 标题缺少官方 Tag,且描述各 section 均为空/占位符,需补充完整内容。
标题建议(可直接复制):
[BugFix] Update PaddleFleet ops import path from paddlefleet.ops to paddlefleet_ops
PR 描述建议(可直接复制,必须复刻 checklist §D2 模板的完整结构):
## Motivation
适配上游 PaddleFleet 包重命名:将 `paddlefleet.ops.deep_ep` 和 `paddlefleet.ops.deep_gemm` 的导入路径分别更新为 `paddlefleet_ops.deep_ep` 和 `paddlefleet_ops.deep_gemm`。
## Modifications
- `fastdeploy/model_executor/layers/moe/ep.py`:更新 deep_ep 导入路径(`paddlefleet.ops.deep_ep` → `paddlefleet_ops.deep_ep`)
- `fastdeploy/model_executor/layers/quantization/fp8_utils.py`:更新 deep_gemm 导入路径(`paddlefleet.ops.deep_gemm` → `paddlefleet_ops.deep_gemm`)
## Usage or Command
N/A
## Accuracy Tests
N/A
## Checklist
- [ ] 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.
- [ ] Format your code, run `pre-commit` before commit.
- [ ] Add unit tests. Please write the reason in this PR if no unit tests.
- [ ] Provide accuracy results.
- [ ] 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.总体评价
变更逻辑清晰,属于上游包路径适配,代码改动正确一致。建议补充 PR 标题 Tag 及描述信息后合入。
CI报告基于以下代码生成(30分钟更新一次): 1 任务总览存在 1 个 Required 失败任务,需优先处理后方可合并。
2 任务状态汇总2.1 Required任务:9/10 通过
2.2 可选任务 — 29/31 通过
3 失败详情(仅 required)Run FastDeploy Unit Tests and Coverage / run_tests_with_coverage — 代码覆盖率不达标(置信度: 高)Run FastDeploy Unit Tests and Coverage / run_tests_with_coverage
根因详情: 关键日志: 修复建议:
关联变更: 链接: 查看日志 |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #7859 +/- ##
==========================================
Coverage ? 63.33%
==========================================
Files ? 462
Lines ? 64365
Branches ? 9871
==========================================
Hits ? 40765
Misses ? 20832
Partials ? 2768
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:
|
Motivation
Modifications
Usage or Command
Accuracy Tests
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.