-
Notifications
You must be signed in to change notification settings - Fork 204
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
in finetune.py 117 Why "-i"?? #32
Comments
The way it is written, it removes filters chosen for pruning, one after another, in a sequential way. Ideally the filters would be removed all at once, that would be a better way of writing this. |
Thank you very much, it turned out to be like this. I just didn’t understand this point. Thanks again.
At 2019-06-24 12:55:56, "Jacob Gildenblat" <notifications@github.com> wrote:
The way it is written, it removes filters chosen for pruning, one after another, in a sequential way.
After the first filter in a layer is pruned, the index of the next filter that needs to be pruned changes, because there is one less filter left.
for example, lets say we need to prune filters 3,10 in some layer.
After pruning filter #3, the layer is smaller, and the index of previous filter #10 is now #9.
Ideally the filters would be removed all at once, that would be a better way of writing this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Can anybody explain that? Thank you very much.
The text was updated successfully, but these errors were encountered: