CycleGAN-gluon-mxnet icon indicating copy to clipboard operation
CycleGAN-gluon-mxnet copied to clipboard

Wrong with ResBlock?

Open CuriousCat-7 opened this issue 6 years ago • 2 comments

The original and advanced types of resnet blocks are list in there: http://torch.ch/blog/2016/02/04/resnets.html

x + R(I(C(R(I(C(x))))) should be changed to R(x + I(C(R(I(C(x)))) )

for

  • R: relu
  • I: InstanceNorm
  • C: Conv2D

CuriousCat-7 avatar Jan 28 '19 05:01 CuriousCat-7

Hi , Sorry for the late reply. i made experiment on Activation and InstanceNorm , but i forgeted to replace the Resblockv2 :(( in this paper : Identity Mappings in Deep Residual Networks it proposed model (e) (ResBlockv2): image

i will change to ResblockV2 x + R(I(C(R(I(C(x))))) = > x + C(R(I(C(R(I(x))))))

thanks your commented

leocvml avatar Jan 31 '19 12:01 leocvml

Hi , Sorry for the late reply. i made experiment on Activation and InstanceNorm , but i forgeted to replace the Resblockv2 :(( in this paper : Identity Mappings in Deep Residual Networks it proposed model (e) (ResBlockv2): image

i will change to ResblockV2 x + R(I(C(R(I(C(x))))) = > x + C(R(I(C(R(I(x))))))

thanks your commented

Thank you for your reply. it makes more sense. However, please note that other famous open sources cyclic gan projects use ResNetBlockV1【x + I(C(R(I(C(x)))) 】, and it seems accepted by others. It would be great to reproduce there method. As far as my experiment, the model acts better after I change the RestNetBlock in the code.

Best regard, Neo

CuriousCat-7 avatar Feb 12 '19 08:02 CuriousCat-7