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

Add ROIWarping layer described in the winning solution of ILSVRC & MSCOCO 2015 competition (http://arxiv.org/abs/1512.04412). #37

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

lim0606
Copy link

@lim0606 lim0606 commented May 4, 2016

Hi, this is Jaehyun Lim

I am currently working on re-implementing the winning solution of ILSVRC & MSCOCO 2015 competition (http://arxiv.org/abs/1512.04412).

As a part of the work, I made the ROI warping layer described in the paper.

Unfortunately, the layer is not complete though. However, I want to share it so that somebody can find the bugs that I couldn't find it until now.......

Note:

  1. The equation (8) in the paper seems to make no sense for me. Thus, I made bilinear warping/mapping as described in Wikipedia.
  2. The backprop (gradient) w.r.t. Image input(or blob) works fine (confirmed with test_jacobian), but the backprop w.r.t. delta_rois (fast-rcnn style rois re-parameterization w.r.t anchors) couldn't make it.

The backprop w.r.t delta_rois are a bit off from the nn.Jacobian.forward. I have tried to found what I might miss, but I couldn't find it until now.

I would appreciate if someone helps me find the bugs.

Best regards,

Jaehyun

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

Successfully merging this pull request may close these issues.

1 participant