-
Notifications
You must be signed in to change notification settings - Fork 71
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
it should be 8x8 not 7x7 #1
Comments
I'm on travel at ML Summer School 2016. I will review it next weekend, sorry for any inconvenience. regards, Jori |
Hello derek4455, I discussed this with one of the engineers in our research team at BSC-CNS, Maurici Yagües, and finally we aggreed the following answer. First of all, let me thanks Maurici for his help in this topic! You are right in pointing out that the explanation and the code are The shrinking is done only on the pooling layers going from 28 in the In the explanation, and in the drawings, the case is described for You can find a more clear explanation in "Chapter 9. Convolutional Thanks for pointing it out, this will be better stated in future |
Thank you very much, that makes sense now! |
From: ShuhaoWang [email protected] I think the padding mode in TensorFlow is pretty tricky. I hope you may explain more about it in your book. Best regards, |
From: Ricky Park: In chapter 5, first convolution layer changes 28x28-->28x28-(pooling)->14x14 but not 28x28-->24x24-->12x12. |
Hey Jordi,
Thanks for the great work. Correct me if i'm wrong (and if so please explain how you got to that number). But on page 114 and in code of cnn.py you write "The resulting output of the convolution has a dimension of 7x7 as we are applying the 5x5 window to a 12x12 space with a stride size of 1" - i assume (because it wasn't mentioned, that padding is 0 - it must be)
But its actually (12-5+0)+1=8 , which is 8x8 not 7x7
Thanks, waiting for your reply!
The text was updated successfully, but these errors were encountered: