Skip to content

Conversation

xingmingyyj
Copy link
Contributor

@xingmingyyj xingmingyyj commented Dec 21, 2023

PR types

Others

PR changes

Others

Description

PIR Op单测修复
修复单测 test_sgd_op_bf16
修复后打开FLAGS_enable_pir_in_executor单测是否通过:修复中
报错信息:.......
遇到的问题:
单测test_sgd_op_bf16.TestSGDOpBF16test_sgd_op_bf16.TestSGDOpBF16Case2
image
新旧IR下执行sgd精度误差过大:
4
原因分析:
造成精度误差过大的原因是因为新旧IR下Kernel选择不一致造成的

  • 旧IR下选择的Kernel
    1
  • 新IR下选择的Kernel
    2

由旧IR输出日志及下面的代码来看,旧IR下如果PADDLE_WITH_DNNL有效,会将数据排布格式由NCHW转化为ONEDNN类型,并且选择对应的Kernel
3

Copy link

paddle-bot bot commented Dec 21, 2023

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Dec 21, 2023
Copy link

paddle-ci-bot bot commented Dec 29, 2023

Sorry to inform you that 3a30f51's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@xingmingyyj xingmingyyj closed this by deleting the head repository Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants