Xiaoxu Chen

Results 12 comments of Xiaoxu Chen

修改PR标题,一句话描述清楚你的工作

参考 `test_distribution_multinomial_static` 增加静态图测试(测试用例保持和动态图一致)

> @cxxly 你好,有两个疑问: > > 1. 请问PR-CI-Static-Check这个CI的错误原因具体是啥? > ![image](https://user-images.githubusercontent.com/46449831/183795148-2cfe1176-5d75-4275-a510-77946f4ee120.png) > 是要人工aprroval才能通过吗?样例代码已经按照标准修改了还是没过。 > 2. 动态图单测里面有个ks test是需要Laplace sample返回一维结果,前提是loc和scale都是0维的,但是静态图里面variable好像无法传入0维参数?其他api的静态图单测没有看到类似的用例,所以这个单测我先注释掉了,辛苦看下是否保留以及需要怎样的修改。https://github.com/PaddlePaddle/Paddle/pull/44913/files#diff-dd8c71861caea3d18f6f55dd112e84257efb9c51d8e7f9d91ef0c85ca5df6e39R238 1) 需要approve 2) 可以先注释掉,备注上注释掉原因。静态图目前还不支持创建零维Tensor,在改进中,

> ![image](https://user-images.githubusercontent.com/46449831/187037435-6fafecc4-c71b-4c11-be5c-b91da2b01087.png) > > @cxxly 单测的时候发现sample函数的采样有时会出现inf的结果,公式和torch以及tensorflow的框架都是一样的,请教一下这是什么原因?已经把采样上下限缩小了eps / 2,还是不能避免这个问题。 这个不太好直接判断,不排除Paddle OP本身可能存在缺陷,需要Debug。 建议 1)在 rsample方法中每一步操作,加上日志,打印每个OP的输入、输出数据 2)因为是随机性问题,重复执行 rsample,直到复现上述问题 3)根据日志定位是写法问题还是OP存在BUG,定位到具体OP 4)如果是写法错误,修改即可;如果是OP存在BUG,提交issue 另外,我在linux机器上没有复现,是只有Windows存在上述问题吗?自己先尝试定位问题,如果无法定位或问题解决不了,再进一步交流

请增加dropout至少一阶反向测试,可以通过固定seed进行数值对齐

需要增加一阶及高阶微分测试用例,具体可以参考test_primapi `sin`

注释写的详细一些,也可以在ReadMe细化,让开发者能够按步骤新增用例

PR标题: A template demonstrating how to test higher-order autodiff.