search

rasd3 / 3D-CVF

[ECCV 2020] This is the official implementation of 3D-CVF: Generating Joint Camera and LiDAR Features Using Cross-View Spatial Feature Fusion for 3D Object Detection
125 stars 23 forks source link

about rbbox_iou_loss #14

Open mc171819 opened 2 years ago

mc171819 commented 2 years ago

hi, when I run 'from spconv.utils import rbbox_iou, rbbox_iou_loss' in box_np_ops.py, it occurs an error: ImportError: cannot import name 'rbbox_iou_loss'. However, when I ask the author about rrbox_iou_loss, he said there isn't any rbbox_iou_loss in spconv. So how should I modify the code?

FANGZHENGFA commented 2 years ago

hi, when I run 'from spconv.utils import rbbox_iou, rbbox_iou_loss' in box_np_ops.py, it occurs an error: ImportError: cannot import name 'rbbox_iou_loss'. However, when I ask the author about rrbox_iou_loss, he said there isn't any rbbox_iou_loss in spconv. So how should I modify the code?

I had same error when I run SECOND repo. The problem may occur in spconv2.x. I solve this problem after installing spconv1.x

mc171819 commented 2 years ago

hi, when I run 'from spconv.utils import rbbox_iou, rbbox_iou_loss' in box_np_ops.py, it occurs an error: ImportError: cannot import name 'rbbox_iou_loss'. However, when I ask the author about rrbox_iou_loss, he said there isn't any rbbox_iou_loss in spconv. So how should I modify the code?

I had same error when I run SECOND repo. The problem may occur in spconv2.x. I solve this problem after installing spconv1.x

rbbox_iou is truly can be import in spconv1.x, however i still cannot import rbbox_iou_loss in any version of spconv. How did you resolve this?

FANGZHENGFA commented 2 years ago

hi I just delete 'rbbox_iou_loss'. Specifically, i change the from spconv.utils import rbbox_iou, rbbox_iou_loss to from spconv.utils import rbbox_iou in box_np_ops.py. I also comment the "riou_cc_loss" function. then it works. Best wish to you!

FANGZHENGFA commented 2 years ago

By the way, my spconv version is v1.0(commit 8da6f96).

I follow this page http://nicethemes.cn/news/txtlist_i198814v.html ,then it works

guzuoxu commented 2 years ago

this help me: https://github.com/idenovix/bu_spconv by the way, when "python setup.py bdist_wheel", there will be a error, it is because the pytorch version, you can solve it follow this: https://blog.csdn.net/baidu_34172099/article/details/109741821