-
Notifications
You must be signed in to change notification settings - Fork 105
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
add Multi-Scale support for WGAN-GP #35
base: master
Are you sure you want to change the base?
Conversation
I 'am not sure this is right or false!! please check it , thank you
# generate random epsilon | ||
epsilon = th.rand((batch_size, 1, 1, 1)).to(fake_samps.device) | ||
# generate random epsilon | ||
epsilon = th.rand((batch_size, 1, 1, 1)).to(fake_samp.device) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a tiny issue here. The epsilon would be different for different real_samp
and fake_samp
, which is not intended; and perhaps also incorrect. Could you add the epsilon as another parameter to this _interp
function and then select an epsilon only once outside? Basically the epsilon should be same for interpolating real and fake samples at all scales.
Rest everything looks good. 👍. Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I will try it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Johnson-yue @akanimax any result of this? This PR is still open?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, I try to adding gp in MSG-GAN, but, it is confusion, the multi-output need gradient for D ,so it can be compute
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I 'am not sure this is right or false!! please check it , thank you