Replace dcu-z100 with test-safety and add test function in batch.py#79297
Replace dcu-z100 with test-safety and add test function in batch.py#79297Ray961123 wants to merge 3 commits into
Conversation
Codecov Report❌ Patch coverage is
❌ Your patch status has failed because the patch coverage (50.00%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## develop #79297 +/- ##
==========================================
Coverage ? 50.00%
==========================================
Files ? 1
Lines ? 2
Branches ? 0
==========================================
Hits ? 1
Misses ? 1
Partials ? 0 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
PaddlePaddle-bot
left a comment
There was a problem hiding this comment.
🤖 Paddle-CI-Agent | pr_review |
2026-06-11 18:00:01
📋 Review 摘要
PR 概述:将 Linux DCU test job runner group 从 dcu-z100 切换为 test-safety,并在 python/paddle/batch.py 添加测试占位函数。
变更范围:.github/workflows/_Linux-DCU.yml、python/paddle/batch.py
影响面 Tag:Execute Infrastructure User Experience
问题
未发现新的阻塞性问题。PR 规范问题在下面章节报,不在这里重复。
历史 Findings 修复情况
| Finding | 问题 | 状态 |
|---|---|---|
| F1 | 生产模块中新增测试占位函数,但没有验证 batch() 的真实行为 |
📝 PR 规范检查
PR 标题缺少官方 Tag,PR 描述未使用仓库要求的四段模板。
标题建议(可直接复制):
[Execute Infrastructure] Replace DCU test runner group and add batch coverage
PR 描述建议(点击展开,可直接复制)
### PR Category
<!-- One of [ User Experience | Execute Infrastructure | Operator Mechanism | CINN | Custom Device | Performance Optimization | Distributed Strategy | Parameter Server | Communication Library | Auto Parallel | Inference | Environment Adaptation ] -->
Execute Infrastructure
### PR Types
<!-- One of [ New features | Bug fixes | Improvements | Performance | BC Breaking | Deprecations | Docs | Devs | Not User Facing | Security | Others ] -->
Improvements
### Description
<!-- Describe what you’ve done -->
- Replace the Linux-DCU test job runner group from `dcu-z100` to `test-safety` in `.github/workflows/_Linux-DCU.yml`.
- Add validation coverage for the `python/paddle/batch.py` module. The current implementation should use a real test under `test/` instead of a production placeholder function.
- Validation: N/A in PR description.
### 是否引起精度变化
<!-- one of the following [ 是 | 否 ]-->
否总体评价
本轮覆盖了 2 个语义变更点:DCU workflow runner group 修改和 batch.py 新增函数。workflow 变更在 diff 内未发现可确定的新缺陷;历史测试占位问题仍未修复,建议改为 test/ 下对 paddle.batch 真实行为的测试覆盖。
CI报告基于以下代码生成(30分钟更新一次): 1 Required任务 : 42/47 通过
2 失败详情🔴 Linux-CPU / Build and test — 环境问题(置信度: 高)分析器: 通用分析(fallback)
关键日志:
修复建议:
关联变更: 未发现与本 PR 修改文件直接关联 🔴 Linux-DCU / Test — PR问题(置信度: 高)分析器: 通用分析(fallback)
关键日志:
修复建议:
关联变更: 🔴 Check — PR问题(置信度: 高)分析器: 通用分析(fallback)
关键日志:
修复建议:
关联变更: PR 描述正文 🟡 Windows-GPU / Build and test — 未知(置信度: 中)分析器: 通用分析(fallback)
关键日志:
修复建议:
关联变更: 未发现与本 PR 修改文件直接关联 🟡 Coverage test — 不稳定问题(置信度: 中)分析器: 通用分析(fallback)
关键日志:
修复建议:
关联变更: 未发现与本 PR 修改文件直接关联 |
Changes
dcu-z100withtest-safetyin.github/workflows/_Linux-DCU.yml_test_batch_placeholderinpython/paddle/batch.py