-
Notifications
You must be signed in to change notification settings - Fork 4
Fix batch norm defaults #83
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
We can also fix them on RETURNN side: rwth-i6/returnn#522 And then maybe via #84. Edit Also done. |
When we just reimplement batch norm here in returnn-common (#89), of course we also should fix them as well. The question is, what defaults to use then. Maybe the new defaults as discussed in rwth-i6/returnn#522. |
The question is also about I would say that See also here for some discussion on RETURNN side. |
We wrapped the RETURNN |
They are wrong in RETURNN. But we can fix them here.
The text was updated successfully, but these errors were encountered: