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

【PPSCI Doc No.24】ppsci.arch.Generator #802

Closed
wants to merge 4 commits into from

Conversation

Turingg
Copy link
Contributor

@Turingg Turingg commented Mar 10, 2024

PR types
Others

PR changes
Docs

Describe
完善 Generator的示例
任务链接:#686
对此模型的具体应用还不熟悉QVQ,阔以看看能这样写嘛

@HydrogenSulfate

Copy link

paddle-bot bot commented Mar 10, 2024

Thanks for your contribution!

@CLAassistant
Copy link

CLAassistant commented Mar 10, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ Turingg
❌ Turingggg


Turingggg seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link

paddle-bot bot commented Mar 10, 2024

❌ The PR is not created using PR's template. You can refer to this Demo.
Please use PR's template, it helps save our maintainers' time so that more developers get helped.

@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Mar 11, 2024
@luotao1 luotao1 closed this Mar 11, 2024
@luotao1 luotao1 reopened this Mar 11, 2024
Copy link
Collaborator

@HydrogenSulfate HydrogenSulfate left a comment

Choose a reason for hiding this comment

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

感谢大佬提交PR,辛苦修改一下

Comment on lines 174 to 175
>>> import warnings
>>> warnings.filterwarnings("ignore", message="When training, we now always track global mean and variance.")
Copy link
Collaborator

Choose a reason for hiding this comment

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

这里的两行应该可以去掉,这类警告应该不会算在doctest的对比内容里

>>> width = 64
>>> input_data = paddle.randn([batch_size, in_channel, height, width])
>>> input_dict = {'in': input_data}
>>> output_data = model(input_dict) # 不需要计算梯度,因为这只是前向传播
Copy link
Collaborator

Choose a reason for hiding this comment

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

"不需要计算梯度,因为这只是前向传播"这句话可以删除

@Turingg Turingg closed this Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants