vickyFox / Region-wise-Inpainting

Region-wise GAN, image inpainting
MIT License
54 stars 17 forks source link

cv2.resize() has wrong order for height/width args #5

Open eridgd opened 4 years ago

eridgd commented 4 years ago

Hi, thank you for sharing such an interesting model.

I noticed that the cv2.resize() calls in test.py have the wrong order for height/width resize args. OpenCV reverses the usual convention for whatever reason.

This line https://github.com/vickyFox/Region-wise-Inpainting/blob/005fe519969f3f2a86647e3a7c7feeb072f8dbdb/test.py#L45

should be:

test_mask = cv2.resize(cv2.imread(file_mask),(args.width, args.height))

and similarly for https://github.com/vickyFox/Region-wise-Inpainting/blob/005fe519969f3f2a86647e3a7c7feeb072f8dbdb/test.py#L52

vickyFox commented 4 years ago

Thank you for your reminder, this is a mistake that we accidentally made while arranging the code, and we will correct the problem immediately. In addition, in our experiment, the width and height are equal, both are 256. The effect will not be affected