liulei01 / DRBox

A deep learning based algorithm to detect rotated object, for example, objects in remote sensing images
Other
421 stars 142 forks source link

About openmp errors in building the code. #1

Open wangzhe0623 opened 6 years ago

wangzhe0623 commented 6 years ago

Hi liulei, thanks for your great job! I downloaded the project and tried to buid it on Ubuntu 14.04. It seems like that you work on Ubuntu 16, so I changed lib "libhdf5-serial" to "libhdf5" in "Makefile" but I got some errors about "openmp", such like "undefined xxxxxx". Then I add "-fopenmp" in "Makefile", code could be compiled. When I "import caffe" with python, still getting errors about openmp. What's wrong with it? I feel so confused, eager for your reply~~~

liulei01 commented 6 years ago

I find that -fopenmp had been added in makefile line 406. How about trying to remove all openmp settings (in Makefile and rbox_util.cpp)and build again? OpenMP is used in an earlier version of my research but is not used in the current version.

wangzhe0623 commented 6 years ago

@liulei01 Thanks for reply~ openmp only involved in rbox_util.cpp? I will try that!

wangzhe0623 commented 6 years ago

Build success~

chl916185 commented 6 years ago

SelectRotatedTarget.m如何使用它制作自己的数据集,能详细的介绍下吗,谢谢啦 @wangzhe0623 @liulei01

xncaffe commented 6 years ago

I compiled your code under ubuntu14.04 system, but there was an error about openmp, I can not solve it myself, asking for your help. Looking forward to your reply. Very grateful! 2018-01-31 16 21 45 @wangzhe0623

wangzhe0623 commented 6 years ago

@xncaffe check out all codes, remove openmp parts~

wangzhe0623 commented 6 years ago

@chl916185 我没有用SelectRotatedTarget.m自己做过哦,我改过labelImg的代码,增加旋转矩形框功能,后来发现增加角度对我用处不大,就没再搞了。

xncaffe commented 6 years ago

我知道的 我一直编译不通过 openmp 那里一直有问题。你当初的问题和我的一样吗?如果一样你能详细帮我解答一下吗?多谢了,我屏蔽了include 但是又有其他错误。谢谢了! 90500a0f 4087ea4e05 9032745a jpg_recompress

xncaffe commented 6 years ago

@wangzhe0623

wangzhe0623 commented 6 years ago

@xncaffe 代码里面调用了openmp,你把它关了就可以了,就是把相关调用的代码注释掉,不加这个并行化一样可以用。

xncaffe commented 6 years ago

好的 多谢