-
Notifications
You must be signed in to change notification settings - Fork 5.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
paddle.static.auc 中ins_tag_weight 有bug #68640
Labels
Comments
问题1修复方案:将ins_tag_weight传入到auc_kernel当中
|
问题2修复方案:遍历ins_tag_weight数据,判断mask是0还是1.
|
如果确定修复方案ok的话,可以考虑往Paddle代码库提相应的PR,到时候我会找相关同学review。 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
bug描述 Describe the Bug
问题1:ins_tag_weight 并没有传给auc kernel中,因此ins_tag_weight 并未生效。
https://github.com/PaddlePaddle/Paddle/blob/release/3.0-beta/python/paddle/static/nn/metric.py
这里的# "InsTagWeight": [ins_tag_weight]为啥没有传进去op里面啊?不传进去怎么判断fake_data?
问题2:auc_kernel里面判断fake_data太粗了,为什么只判断batch里面的第一个数据?
https://github.com/PaddlePaddle/Paddle/blob/release/3.0-beta/paddle/phi/kernels/cpu/auc_kernel.cc
是不是理论上都要判断每个ins_tag_weight_data,从而判断那些mask的数据要丢,那些是不丢的。
其他补充信息 Additional Supplementary Information
No response
The text was updated successfully, but these errors were encountered: