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.

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

Odd numbered pooling height and pooling width gives mismatch #20

Open
Sentient07 opened this issue Feb 24, 2017 · 0 comments
Open

Odd numbered pooling height and pooling width gives mismatch #20

Sentient07 opened this issue Feb 24, 2017 · 0 comments

Comments

@Sentient07
Copy link

Sentient07 commented Feb 24, 2017

I find an ambiguity in the way hstart, hend, wstart, wend are computed. When the pooling height and width are (3,3), the variable bin_size_h and bin_size_w are 2.667 (approximated to three places of decimal).
Then, there are two ways of computing hend. One is the current way, the alternative way being
hend = hstart + bin_size_h
When we compute that way for the dimension that I have mentioned above, (For ph = 1 and roi_start_h = 0. ),
hend = static_cast<int>(ceil(2.667)) + 2;
This gives the output as 5 whereas, the current method of computing in the caffe repository gives the output as 6. Since its a square window, the same applies for width as well.
Could someone please explain this discrepancy?

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

No branches or pull requests

1 participant