Skip to content

【PPSCI Doc No.76】#701

Merged
HydrogenSulfate merged 3 commits into
PaddlePaddle:developfrom
NKNaN:docs-complete
Dec 16, 2023
Merged

【PPSCI Doc No.76】#701
HydrogenSulfate merged 3 commits into
PaddlePaddle:developfrom
NKNaN:docs-complete

Conversation

@NKNaN
Copy link
Copy Markdown
Contributor

@NKNaN NKNaN commented Dec 15, 2023

PR types

Others

PR changes

Docs

Describe

增加了描述和示例代码,看看这样改是否合适,可以的话其他的loss我也按这样来改吧

@paddle-bot
Copy link
Copy Markdown

paddle-bot Bot commented Dec 15, 2023

Thanks for your contribution!

Copy link
Copy Markdown
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.

辛苦大佬

Comment thread ppsci/loss/l1.py Outdated
Comment thread ppsci/loss/l1.py Outdated
Comment thread ppsci/loss/l1.py Outdated
Co-authored-by: HydrogenSulfate <490868991@qq.com>
Co-authored-by: HydrogenSulfate <490868991@qq.com>
@NKNaN
Copy link
Copy Markdown
Contributor Author

NKNaN commented Dec 16, 2023

感谢~

Copy link
Copy Markdown
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.

LGTM

@HydrogenSulfate HydrogenSulfate merged commit e33c326 into PaddlePaddle:develop Dec 16, 2023
@NKNaN NKNaN deleted the docs-complete branch December 26, 2023 14:57
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Feb 20, 2024
huohuohuohuohuo123 pushed a commit to huohuohuohuohuo123/PaddleScience that referenced this pull request Aug 12, 2024
* update docs

* Update ppsci/loss/l1.py

Co-authored-by: HydrogenSulfate <490868991@qq.com>

* Apply suggestions from code review

Co-authored-by: HydrogenSulfate <490868991@qq.com>

---------

Co-authored-by: HydrogenSulfate <490868991@qq.com>
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.

3 participants