-
Notifications
You must be signed in to change notification settings - Fork 766
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
remove api_label
#6142
remove api_label
#6142
Conversation
感谢你贡献飞桨文档,文档预览构建中,Docs-New 跑完后即可预览,预览链接:http://preview-pr-6142.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html |
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~
@SigureMo https://github.com/PaddlePaddle/docs/wiki/飞桨文档相互引用 这里面关于
关于中文 api label,有一下几点思考:
|
我觉得维护这个列表是不合适的,很容易造成不同步
可以的,首先遍历全部 API 文档,找出 label,即可根据「文件路径」->「API 名称」->「英文 label」->「中文 label」得到规范的 label,进而比对即可得知是否规范
首先上面可以统计全部旧 label,并全文搜索旧 label 的引用,进行替换即可 此外,为了规范化,我们应当在中文 API 文档发生改动 / 添加时检查其 label 是否符合新规范 |
@ooooo-create 可以先看看,能不能尝试把中文文档label的问题拆解成一个个任务 |
🙀🙀🙀🙀🙀 2.把 包含 ref 文件,里面 ref 的 label 按照表格查找对应值进行修改 |
感觉我们这个是不需要拆分任务的,应该直接一个 PR 修改就行 这里说的「拆解任务」就是你将整个任务拆成 1、2、3 几个步骤这样就行,已经做的很好了
之后可以考虑上这几部分 |
实操中可能还有考虑一些情况,比如某 api 下的 overview 文档的 label 该怎么定义(如 jit 的 overview 就是
任务要不要拆看 @ooooo-create 师傅叭。像第一步,如果你能直接写个批量替换rst文件第一行标签的脚本,那直接自己做了就OK。像第二步替换 ref ,如果你得手工改并且感觉工作量大的话,放任务也行。(嘿嘿不过还是建议尽量自己尝试一下)
一师傅说的这两点我补充一下
|
看了下有两种一个是overview,还有个少见的en,paddle.tensor里面的overview_en?
自己试试吧
|
可以的,这是一个检查项,即「引用正确性检查」,我们还可以增加一个「label 定义正确性检查」,即检查一个 API 文档第一行的 label 是不是符合规范的,这个检查流程和之前的从「文件路径」-> ... -> 「API label」的检查是一致的,只不过将它放到 CI 进行例行检查 |
嗯
只有一个嘛?一个的话拆出来也是不太好发任务的 😂 |
不可以删的,一个文件两个 label 也是没问题的,因为后面的 label 指向了具体的子 API,可以搜一下,这个 label 是在别的地方有引用的 |
单个label的文件和其引用,我可能(还没有检验)转换好了,我感觉对于多个label的文件,就是判断每行以.. _cn开头,然后寻找接下来第一个copyfrom的那一行的api? 然后这个多label要是在ci中检查的话,只能对于.. _cn格式没错的进行检查, |
我觉得只要检查第一个 label 即可,因为第一个 label 才是整个文件的 label,之后的都是子 API label |
许多文件中的ref: 里面涉及到 fluid的label?,这些需要对应的找api迁移文档,可以开展个任务? |
嗯,我今天还和孙师傅说来着,如果发现有 fluid API 的 label,可以考虑统计下,发个任务,不过不是说带 fluid 的 label 都是 fluid API 的,有些是旧的 API 文档迁移出来但没有规范导致名字里带 fluid 的 对于 fluid API label,可以考虑
当然这个工作量很大,需要拆分一下,可以按照 label 进行拆分 |
可以先初步参考 Paddle 1.8 与 Paddle 2.0 API 映射表,但也不全,得 case by case 的看。如果开发者无法确定的,可以收集起来然后我找建业问,已经和他打过招呼了 |
o 师傅准备好就可以提个 tracking issue 来追踪这些任务了,可以在 tracking issue 里来写一下具体的任务拆分 |
是先把fluid的改好,然后我统一转换其他的吗?还是我先转换,然后再处理fluid? |
我觉得先把非 fluid 改好比较合适吧 |
如 PaddlePaddle/Paddle#56682 所述,英文文档的 label 本就是自动生成的:
docs/docs/api/gen_doc.py
Line 746 in 09e2032
这里会生成一个新的
api_label
,就是按照上述规则自动生成的,repo 里的这个没有任何地方会用到,而且会误导大家已经搜索了下 repo 里的这个
api_label
中不规范的部分(即'api_' + left.replace(".", "_") != right
) 的,发现很少,只有一个remainder_
还没放在paddle.__all__
里无法预览,也在 PaddlePaddle/Paddle#56682 修了另外中文的 api label 可太不规范了,引用 api 必须找那篇文档才知道,以后可以考虑出个小任务来规范一下
PADDLEPADDLE_PR=56682