Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update padding-and-strides_origin.md #1345

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update padding-and-strides_origin.md #1345

wants to merge 1 commit into from

Commits on Jul 30, 2024

  1. Update padding-and-strides_origin.md 

    I think the formula should change \lfloor and \rfloor to \lceil and \rceil. For example, if an (5x5) input performing convolution operation, and the kernel has size of 3x3, strides is 2, the output should be (2x2), but if use "round down", the result is 1x1, we could get right answer if we use "round up".
And if we use the kernel has same size with input, and no padding, strides is 1, the output is 0x0 if we use "round down",but we know it must be 1x1, "round down" could solve this problem.
If my thought has any wrong, hope you could tell me, thanks.
    @NorthOuterTowner
    NorthOuterTowner authored Jul 30, 2024
    Configuration menu
    Copy the full SHA
    9daeba2 View commit details
    Browse the repository at this point in the history