Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not prune the bias #2

Closed
b03902128 opened this issue Apr 5, 2018 · 1 comment
Closed

Not prune the bias #2

b03902128 opened this issue Apr 5, 2018 · 1 comment

Comments

@b03902128
Copy link

Hi Shuan,

Thanks for the great implementation.
I wonder what do you mean by 'didn't prune the bias term'.
Do you mean that you only use Wx (instead of Wx+b) to get the predictions and calculate the gradients?

For the pruned models of interests, should I use:

  1. both new weights and (original) bias (which does not make sense).
  2. only new weights (which may cause negative effects on the accuracy of original models because bia terms are omitted).

Thanks!

@HolmesShuan
Copy link
Owner

To be clear:

  1. Wx refers to without bias term rather than didn't prune the bias term.
  2. We update the bias term during the fine-tuning process, thus the original bias should not be used.
  3. DNS (Dynamic Network Surgery) set some small biases to zero, i.e. prune bias term. Strictly speaking, it is possible that all bias term are set to zero, in another word, Wx+0, which is equivalent to Wx. But we didn't prune bias, even if the value of some bias terms are very close to zero.

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

No branches or pull requests

2 participants