wgan-gp icon indicating copy to clipboard operation
wgan-gp copied to clipboard

mone in generator

Open udion opened this issue 7 years ago • 1 comments

Hi, In the gan training code, since you have already used gen_cost.backward(mone) for generator, why are you still doing gen_cost = -gen_cost

i.e, should it not be

  • just gen_cost.backward(mone) or
  • gen_cost.backward() and then gen_cost = gen_cost*-1 Please correct me if I am wrong.

udion avatar Mar 13 '18 20:03 udion

shouldnt it be 1 instead of -1, the generator should learn to output such a way that the crtiic believes there are real i.e 1 and not -1?

bsridatta avatar Oct 13 '20 15:10 bsridatta