Skip to content

Fix ACGAN training#23

Open
yang-song wants to merge 1 commit into
hwalsuklee:masterfrom
yang-song:patch-2
Open

Fix ACGAN training#23
yang-song wants to merge 1 commit into
hwalsuklee:masterfrom
yang-song:patch-2

Conversation

@yang-song
Copy link
Copy Markdown

The ACGAN training procedure is not optimal

The ACGAN training procedure is not optimal
@ElApseR
Copy link
Copy Markdown

ElApseR commented Feb 6, 2019

I think this is correct and makes training stable.

By the article,

$$L_s=E[logP(S=real|X_{real})]+E[logP(S=fake|X_{fake})]$$ $$L_c=E[logP(C=c|X_{real})]+E[logP(C=c|X_{fake})]$$ $$D = argmax(L_s+L_c)$$ $$G = argmax(L_c-L_s)$$

which means, you don't need to train Q separately. Also, Q shouldn't optimize variables of G.

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

Successfully merging this pull request may close these issues.

2 participants