-
Notifications
You must be signed in to change notification settings - Fork 46
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
Hope for Update to Latest OpenPCDet version #45
Comments
Hello, I do have plan to update it to latest OpenPCDet version, since it can already support different GPU and CUDA version after I update it to openpcdet v0.3. However, we appreciate any contribution for this objective. |
Hi, thanks for your reply! I have tried to use spconv-2.x instead of spconv-1.x, but the performance is lower than your reprted. For example, on nuscens->kitti (SECOND-IoU, with ROS), Car@R11=25.2, Car@R40=21.5 (with spconv-2.x) vs Car@R11=28.3, Car@40=25.4 (with spconv-1.x as you use). I don't know whether the lower performance comes form the higher spconv version? |
Hello, could you present the evaluation log? and have you change the code to suit spconv2.x? |
Hi, the following is the evaluation log. I have changed the code to suit spconv2.x, which follows the implementation in OpenPCDet-0.5.2. Thanks for your help! [2022-06-10 10:20:37,960 test.py 175 INFO] Start logging [2022-06-10 10:22:56,628 eval_utils.py 122 INFO] Result is save to /ssd2/daiyongxing/3D-Detection/ST3D-st3d_v0.3/output/cfgs/da-nuscenes-kitti_models/secondiou/secondiou_old_anchor_ros/default/eval/epoch_50/val/default |
It seems that you have successfully loaded all paramters. Have you checked the evaluation results under spconv 1.x? |
Hi, becuase I have no sudo rights on the server, I can not install spconv 1.x. Spconv 2.x can be installed easily only by pip. Besides, I can not load all paramters of the checkpoint you have provided because the version of Spconv. |
According to the following logging infos, it seems that you have load all parameters.
Latest OpenPCDet should already support load paramters from spconv1.x to spconv 2.x: BTW, you don't need sudo rights to install spconv 1.x. May I know which command stuck your installation of Spconv1.x? Since I don't have suitable code and environment to reproduce this issue intermediately, maybe change spconv to 1.x could be easier to follow in my perspective. |
Thanks for your help. I have successfully loaded parameters from spconv1.x to spconv2.x with your advice. I am still trying to reproduce your results with the latest OpenPCDet with spconv2.x and I found that the num_of_gpus and batch_size_per_gpu could affect the performance significantly on nuscenes->kitti. What are the settings of num_of_gpus and batch_size_per_gpu you used on nuscenes->kitti? |
Hello, we use 8 gpus during the pretrain stage and use 4 gpus in the self-training stage. batch_size_per_gpu is set to 4 for SECOND-IoU and 2 for PVRCNN. |
Hi, do you plan to update this project to the latest OpenPCDet version (e.g., OpenPCDet-0.5+)?
The text was updated successfully, but these errors were encountered: