JUGGHM/PENet_ICRA2021

Confusion about crop size

rightchose opened this issue · 5 comments

The image size is 1216x352? So follow your setting, the image feed in the model with size 1216 x352, but the top of the image is valid. The image I download from kitti depth completion rank.

So I wonder why not use 1216x256?

In my understanding, in main.py the args.val_h and args.val_w mean the size of area we crop from the image using bottom crop, and args.he and args.w is the size crop from the area.

Hi! There are actually two settings. Ours follows the one of Ma et al.(Sparse2Dense) to feed the full sized images into the models. And the other one is cropping the images on the top into 1216x256 ones (e.g. GuideNet). Under this scheme, the top of the images are manually fulfilled when generating results on validation/test sets. Both of them are feasible while there might be some minor difference in performance.

Oh! Thanks a lot. So comparing your model with others on the kitti website, your result may be different on the top of image.