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

Ambiguity regarding roi pooling #36

Open
BornInWater opened this issue Jun 22, 2018 · 0 comments
Open

Ambiguity regarding roi pooling #36

BornInWater opened this issue Jun 22, 2018 · 0 comments

Comments

@BornInWater
Copy link

Hi,
Say i have a region proposal with co-ordinates (top left, bottom right coordinates): (0, 3), (7, 8) and the output i am taking out after roi pooling is 2x2. So, according to the roi pooling code in this repo, we create 4 regions of size([height,width]) [floor(7/2), floor(5/2)],i.e,[3,2]. But in this case, the four regions of size [3,2] would cover only [6,4] area of the region proposal and the features from the rest of the proposal gets ignored.

I came across this blog: https://deepsense.ai/region-of-interest-pooling-explained/ where they select regions like below
screen shot 2018-06-22 at 11 51 51 am

Here, the first box has size [3,2] while the rest of the boxes inside the proposal regions have their sizes adjusted so as to cover and extract features from the entire proposal region.

So, wouldnt the approach mentioned in the above blog be better or is that somehow being ensured in your implementation?

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