Skip to content
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

[Docathon] Fix en api_label #57711

Merged
merged 2 commits into from
Sep 27, 2023
Merged

Conversation

ooooo-create
Copy link
Contributor

@ooooo-create ooooo-create commented Sep 25, 2023

PR types

Others

PR changes

Others

Description

修复英文文档中 ref 引用的api_label
- :ref:`[^a],例如 #57704
- 含有标签的引用
4899d33ef5b7e3b34c52d0972cdabfa

Related links:
#57363

@sunzhongkai588 @SigureMo

@paddle-bot
Copy link

paddle-bot bot commented Sep 25, 2023

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Sep 25, 2023
SigureMo
SigureMo previously approved these changes Sep 25, 2023
Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

其余 LGTMeow 🐾

等孙师傅看看预览~

python/paddle/profiler/profiler_statistic.py Outdated Show resolved Hide resolved
Copy link
Contributor

@sunzhongkai588 sunzhongkai588 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM,就是有个小问题想讨论一下 @SigureMo @ooooo-create

@@ -237,7 +237,7 @@ def googlenet(pretrained=False, **kwargs):
Args:
pretrained (bool, optional): Whether to load pre-trained weights. If True, returns a model pre-trained
on ImageNet. Default: False.
**kwargs (optional): Additional keyword arguments. For details, please refer to :ref:`GoogLeNet <api_paddle_vision_GoogLeNet>`.
**kwargs (optional): Additional keyword arguments. For details, please refer to :ref:`GoogLeNet <api_paddle_vision_models_GoogLeNet>`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

发现了一个问题,ref <api_label> 的时候,api_label是大小写不敏感的?比如 googlenet 这边引用 GoogLeNet,但最后官网生成的引用链接是 .../vision/models/googlenet_en.html#api-paddle-vision-models-googlenet 而不是 .../vision/models/GoogLeNet_en.html @SigureMo

http://preview-paddle-pr-57711.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/en/api/paddle/vision/models/googlenet_en.html

Copy link
Member

@SigureMo SigureMo Sep 26, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这样的话,可能需要对重名的其中一个进行修改了

英文 label 生成见:

https://github.com/PaddlePaddle/docs/blob/f6e6e3a7ab17a06cac2862c4be5d81ca022ab6de/docs/api/gen_doc.py#L745-L747

这个比较类似文件大小写不敏感系统导致的重名问题,可以尝试一个类似的解决方案

中文可能需要同样的解决方案

或许和中文文件名一样使用 _upper 后缀即可,这样甚至中文那边逻辑不需要对 _upper 特判了

@luotao1 luotao1 merged commit 9e2a5a8 into PaddlePaddle:develop Sep 27, 2023
@ooooo-create ooooo-create deleted the fix_en_api_label branch October 8, 2023 05:12
Frida-a pushed a commit to Frida-a/Paddle that referenced this pull request Oct 14, 2023
* fix en api_label

* fix summary
jiahy0825 pushed a commit to jiahy0825/Paddle that referenced this pull request Oct 16, 2023
* fix en api_label

* fix summary
danleifeng pushed a commit to danleifeng/Paddle that referenced this pull request Nov 14, 2023
* fix en api_label

* fix summary
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants