[Metax][CI] update metax ci#7812
Conversation
|
Thanks for your contribution! |
There was a problem hiding this comment.
Pull request overview
该 PR 调整 Metax CI 中 ERNIE-4.5-VL 28B 用例的张量并行配置,将测试从 2 卡 TP 改为 1 卡 TP,以适配单卡运行场景。
Changes:
- 将
tests/metax_ci/test_ernie_28b_vl.py中tensor_parallel_size从 2 改为 1。 - 需要补充 PR 描述中的 Motivation / Modifications,当前仍是模板内容。
- 本次看起来是 CI 配置变更,通常不需要更新用户文档,但建议在 PR 描述中说明无需文档变更。
| cls.llm = LLM( | ||
| model=cls.model_path, | ||
| tensor_parallel_size=2, | ||
| tensor_parallel_size=1, |
CI报告基于以下代码生成(30分钟更新一次): 1 任务总览✅ 所有 Required 任务均已通过,建议合并。
2 任务状态汇总2.1 Required任务 : 2/10 通过
2.2 可选任务 — 25/32 通过
3 失败详情(仅 required)无 required 失败任务。 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #7812 +/- ##
==========================================
Coverage ? 63.15%
==========================================
Files ? 461
Lines ? 64152
Branches ? 9828
==========================================
Hits ? 40516
Misses ? 20860
Partials ? 2776
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:
|
|
/skip-ci ci_iluvatar |
PaddlePaddle-bot
left a comment
There was a problem hiding this comment.
🤖 Paddle-CI-Agent | pr_review |
2026-05-14 15:38:05
📋 Review 摘要
PR 概述:在 Metax CI 测试中取消注释 graph_optimization_config,显式禁用 CUDAGraph 并将 graph_opt_level 设为 0。
变更范围:tests/metax_ci/
影响面 Tag:[Metax] [CI]
📝 PR 规范检查
PR 描述所有段落(Motivation / Modifications / Usage or Command / Accuracy Tests)均为空(仅含占位符注释),Checklist 全部未勾选,不符合描述模板规范。
标题建议(可直接复制):
[Metax][CI] Enable graph_optimization_config in ERNIE-28B-VL Metax CI test
PR 描述建议(可直接复制,必须复刻 checklist §D2 模板的完整结构):
## Motivation
在 Metax CI 的 ERNIE-28B-VL 测试中,`graph_optimization_config` 参数此前被注释掉,导致图优化配置采用默认值。本次变更将其显式启用,设置 `use_cudagraph=False` 和 `graph_opt_level=0`,以提升 Metax 硬件上 CI 测试的稳定性。
## Modifications
- `tests/metax_ci/test_ernie_28b_vl.py`:取消注释 `graph_optimization_config={"use_cudagraph": False, "graph_opt_level": 0}`,在测试 setUp 时显式禁用 CUDAGraph 及图优化。
## Usage or Command
N/A
## Accuracy Tests
N/A(CI 配置变更,不影响模型输出精度)
## 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.
- [ ] Format your code, run `pre-commit` before commit.
- [x] Add unit tests. Please write the reason in this PR if no unit tests.
- [x] 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.问题
未发现阻塞性问题。
总体评价
变更本身简洁合理,显式禁用 CUDAGraph 可提升 Metax CI 稳定性。建议作者补全 PR 描述后合入。
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.