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

【Hackathon 5th No.56】A Physics-Informed Neural Network to solve 2D steady-state heat equation #683

Closed
wants to merge 1 commit into from

Conversation

moosewoler
Copy link

【Hackathon 5th No.56】A Physics-Informed Neural Network to solve 2D steady-state heat equation

@paddle-bot
Copy link

paddle-bot bot commented Oct 6, 2023

你的PR提交成功,感谢你对开源项目的贡献!
请检查PR提交格式和内容是否完备,具体请参考示例模版
Your PR has been submitted. Thanks for your contribution!
Please check its format and content. For this, you can refer to Template and Demo.

f := u_{xx} + u_{yy} \tag{3}
$$

然后通过深度神经网络对$u(x,y)$进行近似。采用Paddle API的$u(x,y)$定义如下:
Copy link
Contributor

Choose a reason for hiding this comment

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

使用英文逗号

Copy link
Author

Choose a reason for hiding this comment

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

已修改

计算域为:

$$
D = \{(x, y)|-1\leq{x}\leq{+1},-1\leq{y}\leq{+1}\}
Copy link
Contributor

Choose a reason for hiding this comment

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

\{ 改为\\{,否则无法正常显示

Copy link
Author

Choose a reason for hiding this comment

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

已修改

Comment on lines +60 to +65
$$
T(-1, y) = 75.0 ^\circ{C} \\
T(+1, y) = 0.0 ^\circ{C} \\
T(x, -1) = 50.0 ^\circ{C} \\
T(x, +1) = 0.0 ^\circ{C}
$$
Copy link
Contributor

Choose a reason for hiding this comment

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

改为

$$
\begin{cases}
T(-1, y) = 75.0 ^\circ{C} \\
T(+1, y) = 0.0 ^\circ{C} \\
T(x, -1) = 50.0 ^\circ{C} \\
T(x, +1) = 0.0 ^\circ{C}
\end{cases}
$$

否则无法正常换行

Copy link
Author

Choose a reason for hiding this comment

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

已修改

u(x,y) + \mathcal{N}[u] = 0 \tag{1}
$$

由式(1)定义$f(x,y)$为:
Copy link
Contributor

Choose a reason for hiding this comment

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

使用英文逗号

Copy link
Author

Choose a reason for hiding this comment

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

已修改

f := u(x,y) + \mathcal{N}[u] \tag{2}
$$

$u(x,y)$可通过深度神经网络进行近似求解。这个假设与方程(2)一起得出了基于物理信息的神经网络$f(x,y)$。这个网络可以通过对组成函数应用链式规则进行自动微分而导出。
Copy link
Contributor

Choose a reason for hiding this comment

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

全文检查一下,公式中使用英文逗号

Copy link
Author

Choose a reason for hiding this comment

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

已修改

Comment on lines +208 to +211
- 202309 : 调研
- 202310 :基于Paddle API的复现
- 202311 :基于PaddleScience的复现
- 202312 :整理项目产出,撰写案例文档
Copy link
Contributor

Choose a reason for hiding this comment

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

这个模型看起来比较简单,按照正常模型复现进度,10月20日左右应该能完成基于PaddleAPI的复现,11月初应该能完成基于PaddleScience的复现;11月中旬完成案例文档。可以参考一下这个进度重新写一下排期规划

Copy link
Author

Choose a reason for hiding this comment

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

受到本职工作影响,此进度过于激进,恐无法完成。

Copy link
Contributor

Choose a reason for hiding this comment

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

辛苦大佬提交RFC文档,建议全文检查一下数学公式渲染的正确性。内联的数学公式两侧的美元符号 $ 均需要空一格(xxx空格$a+b$空格xxx),否则无法在github上正常渲染。

Copy link
Author

Choose a reason for hiding this comment

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

已修改

@luotao1
Copy link
Collaborator

luotao1 commented Oct 18, 2023

已经有 RFC 合入了,如果需要更新和完善,可以在原来的 RFC 上进行修改,感谢!

@luotao1 luotao1 closed this Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants