Skip to content

Conversation

@wanghaoshuang
Copy link
Contributor

@wanghaoshuang wanghaoshuang commented May 27, 2019

  1. Add auto filter pruning strategy.
  2. Add Light-NAS strategy.

1. Fix compressor.
2. Enhence graph executor.
3. Add SAController
4. Add auto pruning strategy.
5. Add unitest for auto pruning strategy.
test=develop
test=develop
test=develop
Copy link
Contributor Author

@wanghaoshuang wanghaoshuang left a comment

Choose a reason for hiding this comment

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

请参考提示进行review

1. Fix unitest on windows.
2. Fix package importing in tests directory.
2. Expose eval_epoch option.
3. Remove unused function in search_agent.
4. Expose max_client_num to yaml file.
5. Move flops constraint to on_epoch_begin function
test=develop
@wanghaoshuang wanghaoshuang force-pushed the slim_ligth_nas branch 3 times, most recently from 57417dc to 734878f Compare June 2, 2019 16:17
2. Remove unitest of light-nas for windows.
test=develop

def update(self, tokens, reward):
"""
Update the controller according to latest tokens and reward.
Copy link
Contributor

@xsrobin xsrobin Jun 6, 2019

Choose a reason for hiding this comment

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

reward->award?不太清楚reward是想表述授予吗?如果是的话,相关的都需要修改

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这里就是需要reward,表示当前搜索出来的网络结构获得的奖励。

Copy link
Contributor

@xsrobin xsrobin left a comment

Choose a reason for hiding this comment

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

请参考对应地方的review意见

XiaoguangHu01
XiaoguangHu01 previously approved these changes Jun 6, 2019
Copy link
Contributor

@XiaoguangHu01 XiaoguangHu01 left a comment

Choose a reason for hiding this comment

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

LG API

xsrobin
xsrobin previously approved these changes Jun 6, 2019
Copy link
Contributor

@xsrobin xsrobin left a comment

Choose a reason for hiding this comment

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

LGTM

test=develop
Copy link
Contributor

@XiaoguangHu01 XiaoguangHu01 left a comment

Choose a reason for hiding this comment

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

LG API

Copy link
Contributor

@xsrobin xsrobin left a comment

Choose a reason for hiding this comment

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

LGTM

@wanghaoshuang wanghaoshuang merged commit 5df65e5 into PaddlePaddle:develop Jun 7, 2019
@wanghaoshuang wanghaoshuang changed the title Add Ligth-NAS for PaddleSlim Add Light-NAS for PaddleSlim Jun 12, 2019
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.

4 participants