search

PaddlePaddle / Paddle

PArallel Distributed Deep LEarning: Machine Learning Framework from Industrial Practice （『飞桨』核心框架，深度学习&机器学习高性能单机、分布式训练和跨平台部署）
http://www.paddlepaddle.org/
Apache License 2.0
22.29k stars 5.61k forks source link

【prim】modify Allclose optest #69709

Closed xiaoguoguo626807 closed 10 hours ago

xiaoguoguo626807 commented 13 hours ago

PR Category

User Experience

PR Types

Bug fixes

Description

pcard-67164

由于allclose的optest kernel_sig 得到的参数是小写的atol, rtol, 导致optest 配置大写的输入不会被正确赋值，一直使用默认的参数。 修改optest 后触发atol, rtol 的string 类型检查，op_compat 中配置的有误。修复后发现optest 传递的atol, rtol 是numpy array 导致api 静态图分支的检查报错，因此删除了相应检查。

最后单测样例中有个x = 10.1 和y = 10 比较， atol=0, rtol=0.01. 不知为什么在动态图和静态图逻辑下虽然用的是同一个kernel， 但是计算的left = x-y = 10.1-10 =0.1, right = atol + rtol * y = 0.1 left-right 却不一致 动态图如下 返回false image

静态图如下返回true , true 为正确。这导致单测检查不过。因此调整了该边界case. image

paddle-bot[bot] commented 13 hours ago

你的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.