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

paddle.Model #4362

Closed
yuyangup opened this issue Mar 25, 2022 · 1 comment · Fixed by #4929
Closed

paddle.Model #4362

yuyangup opened this issue Mar 25, 2022 · 1 comment · Fixed by #4929
Labels
eval docs working group issue

Comments

@yuyangup
Copy link

yuyangup commented Mar 25, 2022

文档中英文链接

中文:https://www.paddlepaddle.org.cn/documentation/docs/zh/develop/api/paddle/Model_cn.html
英文:https://www.paddlepaddle.org.cn/documentation/docs/en/develop/api/paddle/Model_en.html

红线问题

  • fit(train_data=None, eval_data=None, batch_size=1, epochs=1, eval_freq=1, log_freq=10, save_dir=None, save_freq=1, verbose=2, drop_last=False, shuffle=True, num_workers=0, callbacks=None)
    中关于callbacks参数的描述中,建议增加ProgBarLogger 和 ModelCheckpoint这两个默认参数的介绍,以及代码或描述的链接
    少了num_iters参数及对应的说明
  • evaluate(eval_data, batch_size=1, log_freq=10, verbose=2, num_workers=0, callbacks=None)
    少了num_iters参数及对应的说明
  • summary(input_size=None, batch_size=None, dtype=None)
    源码里是没有batch_size这个参数的

结构问题

内容问题

格式问题

主观评价

  • eval_batch(inputs, labels=None)返回:如果没有定义评估函数,则返回包含了预测损失函数的值的列表;如果定义了评估函数,则返回一个元组(损失函数的列表,评估指标的列表)。建议增加一个定义了评估函数的示例
  • predict(test_data, batch_size=1, num_workers=0, stack_outputs=False, callbacks=None) 关于stack_outputs参数的描述不够详细,参考英文版的进行描述
@Ligoml Ligoml added the eval docs working group issue label Mar 25, 2022
RichardWooSJTU pushed a commit to RichardWooSJTU/docs that referenced this issue Apr 8, 2022
@Ligoml
Copy link
Collaborator

Ligoml commented Apr 26, 2022

参考issue中的评估意见,增加更多基础说明与示例

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
eval docs working group issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants