【Hackathon No.48】为 Paddle assign_value、meshgrid、kthvalue、determinant 算子实现 float16 数据类型支持 #52046
Conversation
|
你的PR提交成功，感谢你对开源项目的贡献! |
| ] | ||
|
|
||
| def test_check_grad(self): | ||
| self.check_grad(['Input'], ['Out'], user_defined_grads=self.gt_grad) |
There was a problem hiding this comment.
您好，之前使用user_defined_grads是因为当时fp16的数值梯度计算存在误差，因此使用自定义fp32梯度进行测试。经过验证，目前paddle dev已经修复了精度问题，故取消使用user_defined_grads
| @@ -72,6 +72,16 @@ def init_data(self): | |||
| self.attrs["bool_values"] = [int(v) for v in self.value.flat] | |||
|
|
|||
|
|
|||
| @unittest.skipIf( | |||
| not core.is_compiled_with_cuda(), "core is not compiled with CUDA" | |||
| ) | |||
There was a problem hiding this comment.
这个skip应该可以去掉？现在单测框架能自动为fp16跳过不支持的设备，看后面的其他单测也没有添加这个装饰器。
There was a problem hiding this comment.
好的老师，之前确实其他fp16单测没有加装饰器，但是好像这个单测没有正常跳过，挂掉了。我去掉后重新跑下ci看看。
| @@ -128,5 +138,13 @@ def init_dtype(self): | |||
| self.dtype = "bool" | |||
|
|
|||
|
|
|||
| @unittest.skipIf( | |||
| not core.is_compiled_with_cuda(), "core is not compiled with CUDA" | |||
| ) | |||
There was a problem hiding this comment.
这里的装饰器应该不能去掉，因为它的基类继承自unittest.TestCase，没有自动跳过的机制。
|
关注下PR-CI-Coverage中的单测失败是否和你的修改有关。 |
老师您好，这个我尝试了，官网的nightly包也会挂掉。并且挂掉的这两个单测与我的pr应该没有关系，应该是其他的bug。 |
|
这几个单测目前只在这个PR里挂，其他PR里不会挂。 |
PR types
Others
PR changes
Others
Describe
性能数据（op benchmark）
kthvalue
determinant
assign_value
meshgrid