-
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
No.18 paddle.nn.GLU易用性提升 #58632
No.18 paddle.nn.GLU易用性提升 #58632
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
❌ The PR is not created using PR's template. You can refer to this Demo. |
对应的中文文档也需要补充 |
已经添加中文文档 |
大部分CI已过,能否麻烦老师review @zxcd |
… nnglu resolve conflict
@@ -0,0 +1,76 @@ | |||
# Copyright (c) 2020 PaddlePaddle Authors. All Rights Reserved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
没必要新增一个测试文件,放到这里就可以test/legacy_test/test_glu.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
test/legacy_test/test_glu.py
Outdated
y_var = act(x_var) | ||
y_np = y_var.numpy() | ||
out = glu(self.x, dim) | ||
np.testing.assert_allclose(y_np, out, rtol=1e-05) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里不需要rtol参数吧
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* support paddle.nn.GLU as a layer * add extra_repr ut * add test for type * update doc * update test, merge two test file * delete atol
* support paddle.nn.GLU as a layer * add extra_repr ut * add test for type * update doc * update test, merge two test file * delete atol
PR types
Others
PR changes
Others
Description
#55883