google / compare_gan

Compare GAN code.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

resnet_biggan_deep seems not consist with BigGanDeep in the paper

RuiLiFeng opened this issue · comments

In line 260, resnet_biggan_deep.py, z already be set into tf.concat([z,y],1), while in line 285, when feed parameters to the resnet block, z is treated as the original latents code and y is repeatedly feed into the block, which means that the resnet block in fact get y twice. This seems inconsistent with the settings in the biggan paper.
image
image
image

In this case z and y are indeed the same but only one is used.
We pass around both for consistency with other architectures. Since the only the part in the ResNet block that uses uses z or y is the batch norm, you can configure any of the batch norms and the code should run. I don't think we have batch norm implementation that use both z and y.