Nana

Results 28 comments of Nana

【报名】:7、9-11、31、38

The CI-Coverage check for python code included the last two line of signbit API accidentally.

![image](https://github.com/PaddlePaddle/Paddle/assets/49900969/eda8c55a-0066-4100-b92d-a435145863d7) 从这里看GroupNormGeneralCaseKernel应该是已经支持3D/5D了,通道C在前或者在后都支持,因为这里算imsize的时候,如果数据格式是 [N, C, *],imsize是C之后所有维度的乘积,如果数据格式是 [N, *, C],imsize是N到C之间所有维度的乘积,相当于imsize是除了N和C以外维度的乘积。cuda 的 GroupNormGeneralCaseKernel 和 cpu 的 kernel 都有这一步。后续计算会用到这里的imsize,所以应该除了这个pr修改的地方其他应该都是支持3D/5D的。 这个修改方法是 https://github.com/PaddlePaddle/Paddle/pull/34773 提出的 ![image](https://github.com/PaddlePaddle/Paddle/assets/49900969/df05efad-a545-4d5e-8f49-11b3109eddad) 看记录当时应该是,想支持 [N, C, *]格式的数据,所以做了这个修改,但是同时也支持了 [N, *, C] 现在的文档的数据形状说明这里也是当时修改之后的版本 ![image](https://github.com/PaddlePaddle/Paddle/assets/49900969/e76b7577-c757-4bce-8e49-a3579bb74152) 在现有的 test_group_norm_op_v2.py...

> @NKNaN 好 那就把fp16、bf16下的case支持全吧 fp16、bf16 在 data_layout 是 [N, C, *] 时是通过 GroupNormGeneralCaseKernel 进行计算的,本身是支持3D/5D的。所以应该已经支持全了。 在 test_group_norm_op_v2.py 中增加了 NLC, NHWC, NDHWC (包括fp16数据类型)的测试case,更清楚一些。 bf16的测试在 test_group_norm_op.py 中。

@jeff41404 `paddle.searchsorted` currently does not support fp16 and bf16. I think the reason may be that the two input tensors' data type of `paddle.searchsorted` could be different, and since fp16...

> [the logic of searchsorted](https://github.com/PaddlePaddle/Paddle/blob/develop/paddle/phi/kernels/impl/searchsorted_kernel_impl.h#L105) also supports comparing two inputs with different data types. Registering the fp16 and bf16 data types in the Operator should support these two data types....

@NKNaN 看下这里的覆盖率,如果本地单测有覆盖到,请截图贴上。 这个是 isreal 的内容,不知道为啥这三行也会算到 coverage 检测里面

> 如果可以复用上次的RFC,就不需要改。如果有一些变化,在上次的RFC基础上改就行。 MultivariateNormal 在 paddle 2.6 版本中已有,是否需要继续修改呢? https://www.paddlepaddle.org.cn/documentation/docs/zh/api/paddle/distribution/MultivariateNormal_cn.html#cn-api-paddle-distribution-multivariatenormal