-
Notifications
You must be signed in to change notification settings - Fork 0
Description
Dear Dr. Jiao,
Hi, thank you for sharing your code on GitHub.
I was trying to reproduce your work in my environment and encountered a few issues during the process. I hope you could clarify or provide some missing components.
-
Missing
box_3d_encoder.py
While runningdepth_completion_kitti.py, I encountered an import error related tobox_3d_encoder. Sincedepth_completion_kitti.pyimportsobj_utils, andobj_utils.pyfurther importsbox_3d_encoder, it seems that the corresponding file is not included in the repository.
Could you please provide the requiredbox_3d_encoderimplementation if it is available? -
Missing MobileSAMv2 scripts
In the “Generating Instance Masks” section of the README, the commandbash ./experiments/mobilesamv2.shis mentioned. However, the repository does not contain MobileSAMv2-related scripts or directories under/tools.
Does your project use the vanilla MobileSAM?
If possible, could you also upload the missing script files needed to reproduce your instance mask generation pipeline? -
Depth input discrepancy (
lidar_D80tvslidar_D50t)
In the “Training” step,train_kitti.pyexpects inputs from/lidar,/lidarSAM_t, and/lidar_D50t.
However,depth_completion_kitti.pygenerates/lidar_D80tinstead.
Should the training be performed usinglidar_D80t, or should it follow the defaultlidar_D50tinput expected intrain_kitti.py?
Thank you in advance for your help.
Any clarification or missing files would be greatly appreciated!
Best regards,
Jeonghwa Heo