Skip to content

[Operator Mechanism] Remove debug output from ReduceMeanCheckIfOneDNNSupport#79332

Open
raoyutian wants to merge 5 commits into
PaddlePaddle:developfrom
raoyutian:develop
Open

[Operator Mechanism] Remove debug output from ReduceMeanCheckIfOneDNNSupport#79332
raoyutian wants to merge 5 commits into
PaddlePaddle:developfrom
raoyutian:develop

Conversation

@raoyutian

Copy link
Copy Markdown
Contributor

PR Category

Inference

PR Types

Others

Description

Remove the test output string “ReduceMeanCheckIfOneDNNSupport”

是否引起精度变化

Updated the latest PaddlePaddle release version from 3.2 to 3.3 in the README.
Update the version of PaddlePaddle from 3.2 to 3.3 in the README.
Update the latest release version from 3.0-rc to 3.3 in the README.

@PaddlePaddle-bot PaddlePaddle-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🤖 Paddle-CI-Agent | pr_review | 2026-06-18 09:04:53

📋 Review 摘要

PR 概述:删除 ReduceMeanCheckIfOneDNNSupport 中的调试 stdout 输出。
变更范围paddle/phi/kernels/reduce_kernel_impl.cc 中 OneDNN reduce mean 支持性检查。
影响面 TagOperator Mechanism

问题

未发现阻塞性问题。PR 规范问题在下面章节报,不要在这里重复。

📝 PR 规范检查

标题缺少 Paddle 模板要求的 [Tag];PR 描述四个必填 section 和“是否引起精度变化=否”已填写。

标题建议(可直接复制):

  • [Operator Mechanism] Remove debug output from ReduceMeanCheckIfOneDNNSupport

总体评价

本次代码改动只移除支持性检查函数中的调试输出,不改变 OneDNN fallback 判断、kernel 注册或公开 API。建议合入前补齐标题 Tag,以满足 PR 模板规范。

@raoyutian raoyutian changed the title Remove the test output string ”ReduceMeanCheckIfOneDNNSupport” [Operator Mechanism] Remove debug output from ReduceMeanCheckIfOneDNNSupport Jun 18, 2026
@PaddlePaddle-bot

PaddlePaddle-bot commented Jun 18, 2026

Copy link
Copy Markdown

🤖 Paddle-CI-Agent | ci_status_monitor | 2026-06-18 17:42:58

CI报告基于以下代码生成(30分钟更新一次):
PR commit: 261755d | Merge base: f362de0 (branch: develop)


1 Required任务 : 45/49 通过

总执行(rerun次数) 总任务 ✅ 通过 ❌ 失败 ⏳ 运行中 ⏸️ 等待中 跳过
81(0) 81 77 3 0 0 1
任务 错误类型 置信度 日志
Coverage build PR问题:coverage build size 超阈值待审批 Job
Check approval 需要 Approval Job
Slice / Slice test 不稳定问题:slice benchmark 单 case 性能回退 Job

2 失败详情

🔴 Coverage build — PR问题(置信度: 高)

错误类型: PR问题 | 置信度: 高
分析器: 通用分析(fallback)
失败用例: Coverage build size approval gate

用例 错误摘要
Check coverage build size requires approval coverage build size 增长超过 3G,缺少指定 RD approval

关键日志:

current pr 79332 got approvals: FALSE
This PR make the release paddlepaddle coverage build size growth exceeds 3 G, please explain why your PR exceeds 3G to ext_ppee@baidu.com and in PR description.
Then you must have one RD (swgu98 (Recommend) or luotao1 or risemeup1) approval for this PR
Process completed with exit code 6.
  • 根因摘要: coverage 体积门禁未获 RD approval
    该失败来自 CI 体积审批门禁,不是编译错误。日志明确要求说明 release paddlepaddle coverage build size 增长超过 3G 的原因,并获得 swgu98luotao1risemeup1 中任一 RD approval。

修复建议:

  1. 在 PR 描述中补充 coverage build size 超过 3G 的原因,并请指定 RD 进行 approval;若该体积增长不符合预期,需要先排查构建产物体积来源。

关联变更: PR 仅修改 paddle/phi/kernels/reduce_kernel_impl.cc,删除一行 std::cout,未发现与 coverage 体积增长的直接代码关系。

🔴 Check approval — 需要 Approval(置信度: 高)

该 Job 需要人工 Approval,完成审批后 CI 才会继续执行。

  • 根因摘要: 需要人工审批

修复建议:

  1. 请通过人工审批。
🟠 Slice / Slice test — 不稳定问题(置信度: 中)

错误类型: 不稳定问题 | 置信度: 中
分析器: 通用分析(fallback)
失败用例: Slice benchmark performance check

用例 错误摘要
Getitem - forward - Slice - Slice with Step - float16 - paddle 基线 0.0127795 ms,本次 0.014417920168489218 ms,相对性能提升 -0.12820690703777293,评分 doubt

关键日志:

Getitem - forward - Slice - Slice with Step - float16 - paddle: 基线数据0.0127795, 本次测试数据0.014417920168489218, 相对性能提升-0.12820690703777293, 评分级别doubt
slice测试失败, 存在性能下降case, 失败case性能变化: {'Getitem - forward - Slice - Slice with Step - float16 - paddle': -0.12820690703777293}
Exception: slice测试失败
  • 根因摘要: slice 微基准单 case 性能回退
    日志显示仅一个 Paddle slice micro-benchmark case 低于基线,绝对耗时从约 0.0128 ms 到 0.0144 ms。PR diff 只删除 ReduceMeanCheckIfOneDNNSupport 中的调试输出,未修改 slice/getitem 路径或 benchmark 脚本,因此当前证据更像性能基准波动或环境噪声。

修复建议:

  1. 已知不稳定,请 rerun;若 rerun 后仍稳定复现,再针对 Getitem - forward - Slice - Slice with Step - float16 - paddle 的性能路径单独排查。

关联变更: paddle/phi/kernels/reduce_kernel_impl.cc 删除一行 std::cout;未发现与 slice benchmark 失败 case 的直接关联。

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.

2 participants