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

disA2 and disB2 seem no use in model.py ? #43

Open
LeoXing1996 opened this issue Jan 10, 2020 · 3 comments
Open

disA2 and disB2 seem no use in model.py ? #43

LeoXing1996 opened this issue Jan 10, 2020 · 3 comments

Comments

@LeoXing1996
Copy link

First of all, thanks for your great work !

In your code, disA2 and disB2 are trained via random attribute code, and used to update generator in backward_G_alone.

However, backward_G_alone doesn't seem to be used in training process.

What is the purpose of disA2, disB2 and backward_G_alone ?

@hytseng0509
Copy link
Collaborator

Thank you for your interest! We call the function backward_G_alone here during the training phase.

@LeoXing1996
Copy link
Author

Oh, my fault! I miss this function! But I still confuse about only train disA2 and disB2 via random attribute code and only train disA1 and disB1 via attribute from image.
Why the model need additional discriminators 😕 ?

@HsinYingLee
Copy link
Owner

I think it is first used in BiCycleGAN. It's not for any sound theoretical reasons, but based on experimental results. For high-level explanation, the distribution of translation via random code and via exemplar are quite different, especially in the early stage of training. Using two separate discriminators is more stable and can facilitate the training.

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

3 participants