sefibk / KernelGAN

Other
340 stars 77 forks source link

Using the default setting, the kernel size is 17x17(X2) and 33x33(X4) #51

Open House-Leo opened 4 years ago

House-Leo commented 4 years ago

kernel But in your paper the kernel size should be 13x13.
I look forward to your answer! Thanks!

sefibk commented 4 years ago

You referred to this also in a different issue - please keep them in a single place! Regarding the code - I will have to check if it happens to me too! please verify the packages (torch etc.) are identical to requirements.txt before I look into this. Practically - not how the values in your array are negligible in the boundaries, hence you can simply crop them and you will get an almost IDENTICAL result for the image you are downscaling. That is simply what I did in the paper. I may have a bug in the code that doesn't crop - but it should!

qlawliet commented 3 years ago

kernel But in your paper the kernel size should be 13x13. I look forward to your answer! Thanks!

Hello, I have encountered the same problem as you. May I ask how you finally solve it to make the output kernel size consistent with that in the paper?