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.

Sign up for GitHub

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

Jump to bottom

fix paddle.Model cn docs #4929

Merged
merged 11 commits into from
Jun 17, 2022
Merged

fix paddle.Model cn docs #4929

merged 11 commits into from
Jun 17, 2022

Conversation

SigureMo
Copy link
Member

@SigureMo SigureMo commented Jun 15, 2022 •
edited
Loading

PR 内容

paddle.Model 文档修复

英文文档 PR

PaddlePaddle/Paddle#43537

PADDLEPADDLE_PR=43537

相关 issue

closes PaddlePaddle/Paddle#43343
closes #4252
closes #4362

  • 中文文档的方法标题太乱（16、paddle.Model 文档问题 #4252） fixed in fix api docs style #4426
  • 代码示例没有输出
  • log_freq 默认参数问题
  • model.fit，增加 ProgBarLogger 和 ModelCheckpoint 链接
  • model.fit，增加 num_iters 参数及对应的说明
  • model.evaluate，增加 num_iters 参数及对应的说明
  • model.summary，删除参数 batch_size
  • model.predict，补全 stack_outputs 中文描述
  • model.eval_batch，增加更新示例

@paddle-bot-old
Copy link

感谢你贡献飞桨文档，文档预览构建中，Docs-New 跑完后即可预览，预览链接：http://preview-pr-4929.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html
预览工具的更多说明，请参考：[Beta]飞桨文档预览工具

@SigureMo
Copy link
Member Author

SigureMo commented Jun 15, 2022 •
edited
Loading

关于中文文档标题乱我不太清楚啥意思，没看出来有什么问题……就暂时不改了

@SigureMo SigureMo mentioned this pull request Jun 15, 2022
Merged
@SigureMo
Copy link
Member Author

唔，为什么加了 PADDLEPADDLE_PR=43537 好像也没以那个英文文档 PR 为基础构建？是我操作错了吗？

image

@Ligoml
Copy link
Collaborator

Ligoml commented Jun 16, 2022

关于中文文档标题乱我不太清楚啥意思，没看出来有什么问题……就暂时不改了

这是之前评估的反馈，已经在#4426 完成了修改

@Ligoml
Copy link
Collaborator

Ligoml commented Jun 16, 2022

唔，为什么加了 PADDLEPADDLE_PR=43537 好像也没以那个英文文档 PR 为基础构建？是我操作错了吗？

image

需要等paddle的pr跑完PR-CI-Build才会构建，现在是有的~

Ligoml
Ligoml reviewed Jun 16, 2022
View reviewed changes
docs/api/paddle/Model_cn.rst Outdated Show resolved Hide resolved
docs/api/paddle/Model_cn.rst Show resolved Hide resolved
docs/api/paddle/Model_cn.rst Outdated Show resolved Hide resolved
docs/api/paddle/Model_cn.rst Outdated Show resolved Hide resolved
@SigureMo SigureMo closed this Jun 16, 2022
@SigureMo SigureMo reopened this Jun 16, 2022
Ligoml
Ligoml reviewed Jun 16, 2022
View reviewed changes
docs/api/paddle/Model_cn.rst Outdated Show resolved Hide resolved
docs/api/paddle/Model_cn.rst Outdated Show resolved Hide resolved
docs/api/paddle/Model_cn.rst Show resolved Hide resolved
Ligoml
Ligoml approved these changes Jun 17, 2022
View reviewed changes
Copy link
Collaborator

@Ligoml Ligoml left a comment

Choose a reason for hiding this comment

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

LGTM
good job！

@Ligoml Ligoml merged commit 50c5f25 into PaddlePaddle:develop Jun 17, 2022
@SigureMo SigureMo deleted the fix-model-cn-docs branch June 17, 2022 05:19
@SigureMo SigureMo mentioned this pull request Aug 29, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ligoml Ligoml Ligoml approved these changes

Assignees
No one assigned
Labels
contributor status: not progressed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

paddle.model.fit log_freq 参数不起作用 paddle.Model 16、paddle.Model 文档问题
2 participants
@SigureMo @Ligoml