-
Notifications
You must be signed in to change notification settings - Fork 9
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
Parameters of discriminator optimizer #5
Comments
Ouch, it looks like the parameters of the mpd are never optimized... Unfortunately, I don't have an environment where I can test the fix anymore. If you can compare, could you please comment here with the result? |
Thank you so much for your reply. I'm currently using the fixed code to train my model, but unfortunately I'm working on my own datasets. I may invite my student to do this in the future and keep you updated with the results. |
I understand. Do you see a big difference with your data? |
I think yes since I mainly use URGENT data. |
In open-universe/open_universe/networks/universe/universe_gan.py, line 415 to 417, the code
params_disc = list(self.loss_mrd.parameters()) + list(
self.loss_mrd.parameters()
)
seems to have some bugs.
If I'm correct, it should be:
params_disc = list(self.loss_mrd.parameters()) + list(
self.loss_mpd.parameters()
)
The text was updated successfully, but these errors were encountered: