-
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
【Hackathon 5th No.22】Add CosineAnnealingWarmRestarts API to Paddle -part #57744
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
Sorry to inform you that 04a048e's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually. |
请解决下冲突并通过多条CI |
for place in places: | ||
kwargs = { | ||
'learning_rate': 0.5, | ||
'T_0': 1, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
T_0 = 1
is special case, should add more test case when T_0 > 1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
find a clerical error in rfc, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
python/paddle/optimizer/lr.py
Outdated
When :math:`T_{cur}=0` after restart, set :math:`\eta_t=\eta_{max}`. | ||
|
||
It has been proposed in | ||
`SGDR: Stochastic Gradient Descent with Warm Restarts`_. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
链接没附上吧。写法参考引用方式
python/paddle/optimizer/lr.py
Outdated
eta_min (float, optional): Minimum learning rate. Default: 0. | ||
last_epoch (int, optional): The index of last epoch. Default: -1. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
补充默认值代表的含义
Done |
Sorry to inform you that 4602ef3's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for docs,请补充中文文档
…addlePaddle#57744) * add CosineAnnealingWarmRestarts API * modify test design * modify api design * formate code * add test example * fix documentation
…addlePaddle#57744) * add CosineAnnealingWarmRestarts API * modify test design * modify api design * formate code * add test example * fix documentation
PR types
New features
PR changes
APIs
Description
Add CosineAnnealingWarmRestarts API to Paddle
Link
Rfc PR: PaddlePaddle/community#649
docs PR: PaddlePaddle/docs#6286
待完成