Skip to content

Remove unused cost parameter in ParameterUpdater#970

Merged
reyoung merged 1 commit intoPaddlePaddle:developfrom
reyoung:feature/clean_parameter_updater_finish_pass
Dec 21, 2016
Merged

Remove unused cost parameter in ParameterUpdater#970
reyoung merged 1 commit intoPaddlePaddle:developfrom
reyoung:feature/clean_parameter_updater_finish_pass

Conversation

@reyoung
Copy link
Collaborator

@reyoung reyoung commented Dec 20, 2016

No description provided.

@reyoung reyoung requested a review from qingqing01 December 20, 2016 15:13
@wangkuiyi
Copy link
Collaborator

赞! @reyoung

virtual void startPass() {}

// called by Trainer then finishing a pass, ruturn true if pass accepted
virtual bool finishPass(real cost = 0) { return true; }
Copy link
Member

@jacquesqiao jacquesqiao Dec 21, 2016

Choose a reason for hiding this comment

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

这个 cost一开始是用来做啥的,看起来是记录了每一个batch的cost,现在是使用另外一个机制替代了么

Copy link
Contributor

@qingqing01 qingqing01 left a comment

Choose a reason for hiding this comment

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

查看了下代码,应该只有在已删掉的 BatchRemoteParameterUpdater和 AsgdSparseGradientMachine的finishPass() 中的cost被parameterClient_传送该参数到pserver上,具体作用没有细查看。 目前版本该参数应该可以去掉。

@reyoung reyoung merged commit 4e34220 into PaddlePaddle:develop Dec 21, 2016
@reyoung reyoung deleted the feature/clean_parameter_updater_finish_pass branch December 21, 2016 03:17
zhhsplendid pushed a commit to zhhsplendid/Paddle that referenced this pull request Sep 25, 2019
* break up layers and change index of layers

* prettify script
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

4 participants