Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generator in gan_cifar10 should use BatchNorm1D in preprocess #37

Open
gurgehjm opened this issue Aug 22, 2018 · 1 comment
Open

Generator in gan_cifar10 should use BatchNorm1D in preprocess #37

gurgehjm opened this issue Aug 22, 2018 · 1 comment

Comments

@gurgehjm
Copy link

nn.BatchNorm2d(4 * 4 * 4 * DIM),

As the title says, based on how the code is written, i.e. the size of the noise variable is (BATCH_SIZE, 128), then we should use BatchNorm1d instead of BatchNorm2d.

Is this right?

@wchzh
Copy link

wchzh commented Dec 16, 2018

I have found this problem, and it give me some trouble to find this error. We should use the BatchNorm1d instead of BatchNorm2d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants