-
Notifications
You must be signed in to change notification settings - Fork 5.7k
improve the initializer interface for layers #5632
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
Comments
this initializer for conv2d is designed in purpose, and it's a good initialization method. But it needs to allow users to define their own initializer~ |
For the convolution operators, maybe we need to define a |
I agree with @qingqing01. I have opened an issue for this #5752. |
I think we need to make the initialized method configurable. Even though the |
For example, this paper and its variants. https://www.google.com/url?sa=t&rct=j&q=&esrc=s&source=web&cd=1&ved=0ahUKEwjyxfznscbXAhVQ1mMKHdgDBdwQFggnMAA&url=https%3A%2F%2Farxiv.org%2Fabs%2F1706.02677&usg=AOvVaw2FEBNWmB_5ah6W-73eD2yA In a word, we need to give the right to users. |
@dzhwinter, I am working on a PR to add that flexibility. Will send you all review soon. |
Tasks:
|
The parameters in layers need a better flexible and user-friendly initializer interface. Currently, most of our layers have the fixed init method, such as
conv2d
use thegaussian
initializer conv2d.We need to rewrite that part.
The text was updated successfully, but these errors were encountered: