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
Normalization after data aug #19
Conversation
… the consistency test
@tomMoral the PR is still red because of this import, from a utils file. I don't know how to circumvent this other than reimplementing the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
The normalization that is applied in 3 differents spot is kind of ugly but I think this is good enough and we can improve this in the next iterations.
TODO: get rid of the copy-paste |
Rescaled momentum for SGD tf