-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[Refactoring]Add Sequential with init_weight #884
Conversation
Codecov Report
@@ Coverage Diff @@
## master #884 +/- ##
=======================================
Coverage 66.58% 66.58%
=======================================
Files 145 145
Lines 8828 8841 +13
Branches 1605 1606 +1
=======================================
+ Hits 5878 5887 +9
- Misses 2633 2637 +4
Partials 317 317
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Should we also handle |
As developers always used
nn.Sequential
to implement a submodule, I addSequential
withinit_weight
for parameters initialization withinit_cfg
. In addition, I override__repr__
to display initialization information.example