-
-
Notifications
You must be signed in to change notification settings - Fork 122
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
Fix grad conv im2col #539
Fix grad conv im2col #539
Conversation
I wonder how it is possible this path was never tested. Thank you for this quick fix! Can you add a test? |
I have enabled the tests marked broken that should've checked for this. Let's see if they pass now. |
This reverts commit d648fdd.
Hm those fail for other reasons it seems, added an explicit distinct test. |
035be0a
to
9215fa9
Compare
9215fa9
to
9fdc328
Compare
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.
This LGTM now, the remaining tests are actually fixed broken tests on the CUDA integration side. Will merge tonight unless you have more changes planned.
nope, am all done, go for it! |
Fixes #538
Required for #536