-
Notifications
You must be signed in to change notification settings - Fork 99
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
I really love your work, however #8
Comments
Thank you for your suggestion. Max Pooling 2D is implemented. |
My bad, for mentioning something that was already implemented 😄 |
Yes, it is. However, as I said before I can contribute to the project with
you if you want
…On Thu, Jan 17, 2019 at 7:51 PM Koglin ***@***.***> wrote:
Be aware that you can do BN before activation (e.g. dense, BN, softmax).
And that is done regularly since it gains better results. It's kind of sad
that the model summary does not incorporate that important information.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#8 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AY207fzesDw7RQbJLfLJh9Mx-ESPClUeks5vEMZHgaJpZM4ZkaQB>
.
|
I really love your work, as I said earlier but why don't you add the other layers, (Batch Normalization, Activation, Dropout, Zero Padding 2D, Max Pooling 2D), or at least package your code?
I am ready and willing to contribute with you, if you want..
The text was updated successfully, but these errors were encountered: