-
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
[xdoctest] reformat example code with google style in No.65-68 #55953
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
model.py 的 save原始示例代码有问题,所以重写了,改动较大。 |
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.
有个笔误,其他 OK ~ 辛苦 ~:)
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.
LGTMeow 🐾
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.
python/paddle/hapi/hub.py
Outdated
repo_dir(str): Github or local path. | ||
github path (str): A string with format "repo_owner/repo_name[:tag_name]" with an optional | ||
tag/branch. The default branch is `main` if not specified. | ||
local path (str): Local repo path. |
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.
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.
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.
@enkilee sphinx 语法务必 十分注意 空行和缩进。
源码中一句话换行时,需要和上一行对齐(比如这里的tag/branch 和 github 对齐),这样前端显示会仍旧保持是连着的一句话。
另外,每个缩进都代表一个层级,请注意~
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.
收到
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.
谢谢各位大佬解惑
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.
赞!!!
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 ~ 赞!
PR types
Others
PR changes
Others
Description
预览:
en: http://preview-paddle-pr-55953.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/en/api/paddle/abs_en.html
zh: http://preview-paddle-pr-55953.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/paddle/abs_cn.html
关联 PR:
#55629
#55295
@sunzhongkai588 @SigureMo @megemini