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

Bug with tools/test.py, when no 3dbox, the program can't run. #1198

Closed
MalignusCN opened this issue Jan 26, 2022 · 2 comments · Fixed by #1212
Closed

Bug with tools/test.py, when no 3dbox, the program can't run. #1198

MalignusCN opened this issue Jan 26, 2022 · 2 comments · Fixed by #1212
Assignees

Comments

@MalignusCN
Copy link

Hi, I just found that when I run tools/test.py with your pretrained model as follow command:
python tools/test.py /wangyh12/mmdetection3D/configs/fcos3d/fcos3d_r101_caffe_fpn_gn-head_dcn_2x8_1x_nus-mono3d.py /wangyh12/mmdetection3D/checkpoints/fcos3d_r101_caffe_fpn_gn-head_dcn_2x8_1x_nus-mono3d_20210715_235813-4bed5239.pth --eval mAP --show --show-dir/wangyh12/mmdetSave
It turns out that with pic '../nuScenes/samples/CAM_BACK_RIGHT/n015-2018-08-02-17-16-37+0800__CAM_BACK_RIGHT__1533201470427893.jpg', the model can't predict a 3dbox, which with your code in mmdet3d/core/bbox/structures/cam_box3d.py. There is a command in 129 lines, to assert that there must be at least one 3dbox. I'm afraid that this should be a bug, plz help to fix that problem. Thx

@ZCMax
Copy link
Collaborator

ZCMax commented Feb 7, 2022

Sorry for the late reply, could you please offer the error log so that we could fix it ASAP?

@ZCMax
Copy link
Collaborator

ZCMax commented Feb 7, 2022

After rechecking the code, the assertion problem in 129 lines have been modified in v1.0.0.dev0 branch, we will fix the bug in master branch in the future.

@wHao-Wu wHao-Wu linked a pull request Feb 8, 2022 that will close this issue
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 a pull request may close this issue.

4 participants