-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
python config parser重构之optimizer #1210
Comments
下面这段代码里看上去有一些不一致性
我了解这些不一致性,有很多和 @jacquesqiao 的设计没关系,是因为历史原因。但是在我们的v2里看起来必须fix。 |
zhhsplendid
pushed a commit
to zhhsplendid/Paddle
that referenced
this issue
Sep 25, 2019
* refine the fill_constant doc test=develop * fix according the standard test=develop * fix fill_constant doc test=develop * fix from standard test=develop
lizexu123
pushed a commit
to lizexu123/Paddle
that referenced
this issue
Feb 23, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
为了提供更加灵活的python使用方法,在之前的工作 #1108 中,开始进行python api v2的开发工作,抽象出来了 layers,optimizer,data等组件。
之前的重构是对原有的config_parser.py 做了一层封装,对于这种封装的改造方式,王益老师的意见是:
按着这种思路,在上述的几个组件中感觉optimizer这个组件相对比较独立,所以决定第一步先把optimizer和相关配置独立出来。
主要方案:
这样做的好处:
使用方式:
The text was updated successfully, but these errors were encountered: