Skip to content

Conversation

pengli09
Copy link
Contributor

These three parameters have already been moved to model_average. Leaving them here will cause duplicate keys in kwargs.

…imizers.py

These three parameters have already been moved to model_average. Leaving
them here will cause duplicate keys in kwargs

args = [
'batch_size', 'learning_rate', 'learning_rate_decay_a',
'learning_rate_decay_b', 'learning_rate_schedule', 'learning_rate_args',
Copy link
Collaborator

Choose a reason for hiding this comment

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

看起来,这里多了个,
所以导致Paddle格式检查不过。

目前paddle使用pre-commit来做格式检查。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@reyoung reyoung merged commit 638cf8d into PaddlePaddle:develop Dec 15, 2016
@pengli09 pengli09 deleted the fix-model-avg-py-error branch December 20, 2016 08:17
@pengli09 pengli09 restored the fix-model-avg-py-error branch December 20, 2016 08:22
@wangkuiyi wangkuiyi mentioned this pull request Mar 2, 2017
wangxicoding pushed a commit to wangxicoding/Paddle that referenced this pull request Dec 9, 2021
lizexu123 pushed a commit to lizexu123/Paddle that referenced this pull request Feb 23, 2024
zhangyuqin1998 pushed a commit to zhangyuqin1998/Paddle that referenced this pull request Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants