tf_unet icon indicating copy to clipboard operation
tf_unet copied to clipboard

Padding Options

Open weezhao96 opened this issue 8 years ago • 3 comments

Hi jakeret,

I have been using your U-Net implementation for a few months now. Thank you for sharing it out! Do you see an issue if I try to implement a 'SAME' padding for convolutions for this specific U-Net network?

Zhao

weezhao96 avatar Sep 05 '17 02:09 weezhao96

Great to hear that the package is useful to you. Any kind of contribution is highly welcomed.

In principle I don't see a problem. Ideally, the padding becomes a parameter where the default is the current behaviour.

Looking forward to your Pull Request

jakeret avatar Sep 05 '17 06:09 jakeret

Isn't adding padding during segmentation might decrease the segmentation accuracy? I could not find any reference for this, but presumably, when we do zero-padding, we add some information that is not present in the original image. What do you think?

BAILOOL avatar Sep 05 '17 06:09 BAILOOL

For the maxpooling the padding shouldn't have an impact. For the conv/deconv this could indeed introduce a bias but i'm not sure how big the impact would be

jakeret avatar Sep 05 '17 13:09 jakeret