EDVR icon indicating copy to clipboard operation
EDVR copied to clipboard

about DCNv2 compiling

Open dawnlh opened this issue 4 years ago • 4 comments

In the Dependencies and Installation part, you mentioned that:

Deformable Convolution. We use Charles Shang's DCNv2 implementation. Please first compile it.

cd ./codes/models/modules/DCNv2
bash make.sh

but there is no "modules" directory in this path, so do you mean I should download it from 'Charles Shang's DCNv2' repository and put it in this path then compile it myself? (if I run the code directly, I will encounter the error--"Failed to import DCNv2 module")

Thanks a lot

dawnlh avatar Sep 17 '19 04:09 dawnlh

In the Dependencies and Installation part, you mentioned that:

Deformable Convolution. We use Charles Shang's DCNv2 implementation. Please first compile it.

cd ./codes/models/modules/DCNv2
bash make.sh

but there is no "modules" directory in this path, so do you mean I should download it from 'Charles Shang's DCNv2' repository and put it in this path then compile it myself? (if I run the code directly, I will encounter the error--"Failed to import DCNv2 module")

Thanks a lot

Hi, you can change to the 'dev' branch, then you will find "modules" directory.

townblack avatar Sep 20 '19 07:09 townblack

@dawnlh We have updated the repo to use dcn in mmdetection. But we did not update the readme, so it raised the confusion. Now we have updated the readme.

xinntao avatar Sep 20 '19 08:09 xinntao

@xinntao Is there any difference between using DCNv2 and DCN? DCNv2 performs better?

dawnlh avatar Sep 20 '19 13:09 dawnlh

@townblack oh, I got it, thanks!

dawnlh avatar Sep 20 '19 13:09 dawnlh