Skip to content
This repository has been archived by the owner on Jun 8, 2023. It is now read-only.

Error in Figure 4.7 (Page 27) Description? #18

Open
ShikharJ opened this issue May 4, 2018 · 2 comments
Open

Error in Figure 4.7 (Page 27) Description? #18

ShikharJ opened this issue May 4, 2018 · 2 comments
Labels

Comments

@ShikharJ
Copy link

ShikharJ commented May 4, 2018

Thanks guys for this amazing report, it helped me a lot in understanding convolutional arithmetic.

I think I found a slight error in the description of the figure 4.7 description on page 27 (line 3). Straight from the report:

The transpose of convolving a $3 \times 3$ kernel over a $6 \times 6$ input padded with a $1 \times 1$ border of zeros using $2 \times 2$ strides (i.e., $i = 6$, $k = 3$, $s = 2$ and $p = 1$). It is equivalent to convolving a $3 \times 3$ kernel over a 2 \times 2 input (with $1$ zero inserted between inputs) padded with a $1 \times 1$ border of zeros (with an additional border of size $1$ added to the bottom and right edges) using unit strides (i.e., $i' = 3$, $\tilde{i}' = 5$, $a = 1$, $k' = k$, $s' = 1$ and $p' = 1$).

Shouldn't this be 3 \times 3 ? The visual is probably correct, this feels erroneous, and can be seen by calculating directly from the formula of the direct convolution.

@jianchao-li
Copy link

jianchao-li commented Sep 14, 2018

Just found the same mistake as did @ShikharJ. It appeared in the caption of Figure 4.7 on Page 27, shown below.
mistake
The text with a red underline should be 3 x 3 input.

@fvisin
Copy link
Collaborator

fvisin commented Oct 8, 2018

Thank you for reporting this typo. It should indeed read 3 x 3 input, we will fix it along with the rest in the next update to the guide.

Thank you again for reporting it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants