search

oeway / pytorch-deform-conv

PyTorch implementation of Deformable Convolution
MIT License
911 stars 151 forks source link

What is the size of the offset？ #14

Closed guoswang closed 5 years ago

guoswang commented 6 years ago

Hi, I have read the paper“Deformable Convolutional Networks”and your pytoch-deform-conv code. Then I think that the size of offset maybe bckenel[0]kenel[1]h*w. Maybe I understand it wrong. Can you explain it?

aurora95 commented 5 years ago

@guoswang I think you understand it correctly... This repo makes a wrong implementation

oeway commented 5 years ago

hi, @aurora95 thanks for your reminder. Unfortunately I don’t have time to work on this right now.

However if you could give some more details about the issue, or ideally a PR would be great for people who use this repo. Thanks.

Originally this repo is ported from the tensorflow implementation, I didn’t any further analysis than the original repo.

On Fri 14 Sep 2018 at 05:35, aurora95 notifications@github.com wrote:

@guoswang https://github.com/guoswang I think you understand it correctly... This repo makes a wrong implementation

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/oeway/pytorch-deform-conv/issues/14#issuecomment-421220831, or mute the thread https://github.com/notifications/unsubscribe-auth/AAdNyxeGR8z5zO4BVd5vrTljFWTzkPxEks5uayP8gaJpZM4TDZ5l .

guoswang commented 5 years ago

@aurora95 But I still want to say tha thanks for your great work !!