-
Notifications
You must be signed in to change notification settings - Fork 874
[API Compatibility] Add nn.Constant/Reflection/Replication Pad layers #7596
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
base: develop
Are you sure you want to change the base?
Conversation
|
感谢你贡献飞桨文档,文档预览构建中,Docs-New 跑完后即可预览,预览链接:http://preview-pr-7596.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html |
📚 本次 PR 文档预览链接(点击展开)
|
ooooo-create
left a comment
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.
这个的流水线是效率云,这些符号是不是显示不出来。另外之前大小写敏感的解决方式是在大写的那个 加上 __upper, 比如 https://github.com/PaddlePaddle/docs/blob/develop/docs/api/paddle/metric/Accuracy__upper_cn.rst 和 https://github.com/PaddlePaddle/docs/blob/develop/docs/api/paddle/metric/accuracy_cn.rst.
LGTM for check_api_label_cn.py change
ooooo-create
left a comment
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.
Paddle 里1D 的 D 大写才是正统,小写的 API 才是兼容性的 API,这个也走 upper的方案吗?还是小写用 __lower表示? |
也走 |
sed lower refine reference label in lower
7729c8a to
0173528
Compare


1. 新增如下 API 文档:
nn.ConstantPad1D/2D/3D
nn.ReflectionPad1D/2D/3D
nn.ReplicationPad1D/2D/3D
nn.CircularPad1D/2D/3D
以及上述十二个 API "D" 为小写的别名(例如 nn.ConstantPad1D -> nn.ConstantPad1d)。
由于 rst 的标签不区分大小写,同时避免在不区分大小写的文件系统内文件重名的问题,小写的 API 文档增加了
compat后缀,标签也增加了compat后缀。关联 PR:PaddlePaddle/Paddle#76136
2. 优化根据文件名校验首行 Label 的逻辑,展示所有报错信息后再退出,并在错误信息中打印预期值。
旧报错:
新报错: