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

Is it necessary to use use a different class than the DCGAN class to build GAN #19

Open
piyush01123 opened this issue Feb 4, 2019 · 0 comments

Comments

@piyush01123
Copy link

I was trying to implement https://github.com/roatienza/Deep-Learning-Experiments/blob/master/Experiments/Tensorflow/GAN/dcgan_mnist.py and it works fine. But I tried the exact same code without the wrapper class ie I wrote just one class DCGAN and tried to build D, G and AM in the same class through various functions. I came across very weird Keras errors like TypeError: __call__() missing 1 required positional argument: 'inputs' for the line

. Does this mean the Keras's Sequential object is stateful and changes usage based on whether it is used in the same class or different one?

Also, I noticed that there's a lot of duct-tape code like

 if self.D:
    return self.D

This pattern is repeated whenever you create a D/G/AM model. Why is this. Is this related to the other questions I raised?

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

1 participant