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

Feature Request: Add a new section in Model Zoo for Generation #489

Open
rahulsnkr opened this issue Dec 5, 2018 · 8 comments
Open

Feature Request: Add a new section in Model Zoo for Generation #489

rahulsnkr opened this issue Dec 5, 2018 · 8 comments

Comments

@rahulsnkr
Copy link

Generative Adversarial Networks (GANs) have gained a lot of traction, both from research point of view, as well as development, due to their numerous applications. A lot of GAN architectures have also been published, for example -

Therefore, it makes sense to have a separate section for Generation in the Model Zoo, and GAN architectures could be added within that section

I'd like to work on this, starting with Vanilla GAN

@Ram81
Copy link
Member

Ram81 commented Dec 5, 2018

@rahulsnkr sure go ahead

@rahulsnkr
Copy link
Author

rahulsnkr commented Dec 5, 2018

At the moment, Fabrik is not capable of importing/exporting two networks at the same time, and the given issue is dependent on the same, since a GAN consists of two networks, i.e. a generator and a discriminator. How should I proceed from here? Should I try to add support for Fabrik so that it is capable of supporting more than one network? @Ram81

@thatbrguy
Copy link
Contributor

@rahulsnkr In my opinion supporting more than one network does not sound like a good idea. Other than massive UI redesign, we also have to take care of proper usage of graphs and sessions in Keras and TF.

Instead of that, we can just use Generators and Discriminators separately. To be honest, most Generators are variants of U-Nets and Autoencoders. Similarly, Discriminators are straightforward too, except for some like the PatchGAN. Keeping Generators and Discriminators separate would also allow you to mix and match combinations.

What you can do instead is while exporting a Generator we could have a popup that allows you to download a Discriminator as well. We can also have toy code samples to enable loading a GAN and using them. @Ram81 What do you think?

@rahulsnkr
Copy link
Author

@thatbrguy yes that seems like a much better idea. It will also give the users more flexibility. Maybe the 'Generation' section can have two subsections for generators and discriminators, and as you said, the user can be prompted to download a discriminator when exporting a generator, and vice versa. @Ram81 does this sound ok?

@kartikdutt18
Copy link

@rahulsnkr @Ram81 If generators and discriminators are made available separately, Can I work on DCGAN ?

@Ram81
Copy link
Member

Ram81 commented Dec 10, 2018

@rahulsnkr can you research if it's feasible to have multiple model in fabrik's current implementation. If it is not feasible then we can go ahead with some other approach

@rahulsnkr
Copy link
Author

@Ram81 i'll look into it!

@rohanasthana
Copy link

rohanasthana commented Jan 6, 2019

Is this still open? I would like to work on this issue

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

5 participants