PENet_ICRA2021 icon indicating copy to clipboard operation
PENet_ICRA2021 copied to clipboard

Confusion about crop size

Open rightchose opened this issue 3 years ago • 5 comments

0000000000

rightchose avatar Sep 02 '21 13:09 rightchose

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.

rightchose avatar Sep 02 '21 13:09 rightchose

So I wonder why not use 1216x256?

rightchose avatar Sep 02 '21 13:09 rightchose

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.

rightchose avatar Sep 02 '21 13:09 rightchose

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.

JUGGHM avatar Sep 02 '21 13:09 JUGGHM

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

rightchose avatar Sep 03 '21 03:09 rightchose