Skip to content

support config list which length is 1 and add “model.backend" for pruning and distillation config#310

Merged
chensuyue merged 3 commits intomasterfrom
penghuic/config_list
Dec 20, 2022
Merged

support config list which length is 1 and add “model.backend" for pruning and distillation config#310
chensuyue merged 3 commits intomasterfrom
penghuic/config_list

Conversation

@PenghuiCheng
Copy link
Copy Markdown
Contributor

@PenghuiCheng PenghuiCheng commented Dec 20, 2022

Signed-off-by: Cheng, Penghui penghui.cheng@intel.com

Type of Change

Enhancement API
No API changed

Description

support config list which length is 1
set model.backend for "default" in new API
fixed distillation criterion is none issue for oneshot optimization.

Expected Behavior & Potential Risk

we can pass a list with one config to prepare_compression

How has this PR been tested?

local tested

Signed-off-by: Cheng, Penghui <penghui.cheng@intel.com>
Signed-off-by: Cheng, Penghui <penghui.cheng@intel.com>
@PenghuiCheng PenghuiCheng changed the title support config list for only one in the list support config list for only one in the list and add “model.backend" for pruning and distillation config Dec 20, 2022
Signed-off-by: Cheng, Penghui <penghui.cheng@intel.com>
@PenghuiCheng PenghuiCheng changed the title support config list for only one in the list and add “model.backend" for pruning and distillation config support config list which length is 1 and add “model.backend" for pruning and distillation config Dec 20, 2022
@chensuyue chensuyue merged commit 6a6e2de into master Dec 20, 2022
@chensuyue chensuyue deleted the penghuic/config_list branch December 20, 2022 15:05
VincyZhang pushed a commit that referenced this pull request Feb 12, 2023
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.

2 participants