Skip to content
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

[WIP] All test errors #60488

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Conversation

zrr1999
Copy link
Member

@zrr1999 zrr1999 commented Jan 1, 2024

PR types

Others

PR changes

Others

Description

全部添加 @test_with_pir_api

@paddle-bot paddle-bot bot added the contributor External developers label Jan 1, 2024
fix type

fix

fix

fix

fix
@zrr1999
Copy link
Member Author

zrr1999 commented Jan 4, 2024

image

类型支持情况不统一

部分静态图的类型检查有滞后(或支持情况不一致),跟实际 PIR 支持的类型不一致。
比如上面的例子,PIR中是支持int32的,所以并不会报错。

image 部分静态图支持的类型 PIR 不支持,例如上面的softmax。

解决方案

限制部分代码只在非PIR情况下测试,比如改成

with program_guard(Program(), Program()):
            ...
            if not in_pir_mode():
                self.assertRaises(TypeError, paddle.mean, input2)
            ...

@zrr1999
Copy link
Member Author

zrr1999 commented Jan 4, 2024

image

不存在适用 PIR 的用例

比如上面的test_errors,除了 动态图的部分就是 PIR 已经支持的,所以添加是没有用的。

@zrr1999
Copy link
Member Author

zrr1999 commented Jan 16, 2024

image 段错误 dropout、full_like

@zrr1999
Copy link
Member Author

zrr1999 commented Jan 16, 2024

类型不支持,可通过注册类型修复
image

例如test_lookup_table_v2_op.py

@0x45f
Copy link
Contributor

0x45f commented Jan 17, 2024

image #### 类型支持情况不统一 部分静态图的类型检查有滞后(或支持情况不一致),跟实际 PIR 支持的类型不一致。 比如上面的例子,PIR中是支持int32的,所以并不会报错。

image 部分静态图支持的类型 PIR 不支持,例如上面的softmax。

解决方案

限制部分代码只在非PIR情况下测试,比如改成

with program_guard(Program(), Program()):
            ...
            if not in_pir_mode():
                self.assertRaises(TypeError, paddle.mean, input2)
            ...
  • 对于mean:这个case我的理解是mean kernel是支持int32类型的,但是因为老静态图python的检查滞后所以检查的时候会报错。这里正确的做法是在老静态图python端dtype检查中加上int32类型,然后把这个测试case给删掉。因为kernel是支持int32类型的,这里的测试case是不合理的~
  • 对于softmax:cpu softmax kernel没有注册float16类型,但是gpu softmax kernel注册了float16类型,对于softmax测试case争取的修改方式是判断一下是否paddle.is_compiled_with_cuda() ,参考test/legacy_test/test_activation_op.py中TestSinhOpError的处理方式~

NOTE:怎么看kernel是否注册了某个dtype,以softmax为例。在paddle/phi/kernels/cpu/softmax_kernel.cc 和 paddle/phi/kernels/gpu/softmax_kernel.cu 分别有cpu和gpu注册的dtype类型~

@0x45f
Copy link
Contributor

0x45f commented Jan 17, 2024

image #### 不存在适用 PIR 的用例 比如上面的test_errors,除了 动态图的部分就是 PIR 已经支持的,所以添加是没有用的。

对于mean的处理上面已经回复过~

@0x45f
Copy link
Contributor

0x45f commented Jan 17, 2024

类型不支持,可通过注册类型修复 image

例如test_lookup_table_v2_op.py

  • 这里不建议给kernel新增注册类型,这里的处理方法应该和上面提到的softmax类似,可以看下uniform cpu和gpu注册的dtype~

@0x45f
Copy link
Contributor

0x45f commented Jan 17, 2024

image 段错误 dropout、full_like

  • 段错误的我来看下吧,这样看不出具体是什么问题~

Copy link

paddle-ci-bot bot commented Jan 18, 2024

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

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