Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

fix total sparsity in sa & auto compress pruner bug #4474

Merged
merged 4 commits into from
Jan 18, 2022

Conversation

J-shang
Copy link
Contributor

@J-shang J-shang commented Jan 18, 2022

Description

Add warning to SA and auto compress pruner if user set sparsity or sparsity_per_layer.
Fix example & ut bug.
Fix op_names not expand in SA task generator.

@J-shang J-shang changed the title fix total sparsity in sa & auto compress pruner fix total sparsity in sa & auto compress pruner bug Jan 18, 2022
@J-shang J-shang changed the base branch from master to v2.6 January 18, 2022 04:23
# TODO: replace with validation here
for config in config_list:
if 'sparsity' in config or 'sparsity_per_layer' in config:
_logger.warning('Only `total_sparsity` can be differentially allocate sparse ratio to each layer, `sparsity` or `sparsity_per_layer` will allocate fix sparse ratio to layers. Make sure you know what this will lead to, otherwise please use `total_sparsity`.')
Copy link
Contributor

Choose a reason for hiding this comment

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

There is a grammar problem, please fix it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

allocate -> allocated, fix -> fixed

@liuzhe-lz liuzhe-lz merged commit fd8fb78 into microsoft:v2.6 Jan 18, 2022
@J-shang J-shang deleted the bug_fix branch January 25, 2022 05:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants