-
Notifications
You must be signed in to change notification settings - Fork 15
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
Refactor Connection Sparsity #240
base: dev
Are you sure you want to change the base?
Conversation
connection sparsity can be calculated directly looking at the name of the parameters in the state dict
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #240 +/- ##
==========================================
+ Coverage 64.69% 65.36% +0.67%
==========================================
Files 16 16
Lines 796 745 -51
Branches 161 147 -14
==========================================
- Hits 515 487 -28
+ Misses 239 221 -18
+ Partials 42 37 -5
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Also consider the biases, like in RNN. And should track parameters named with 'bias' |
This pull request introduces an extension to calculate the connection sparsity of the model.
From now on, it is possible to calculate the connection sparsity even though the model doesn't have layers supported by the documentation list.