Closed tianyu-z closed 4 years ago
I know this is WIP but be careful: you have a weird .bak
file which should not be committed and not pushed + you are overwriting the current defaults.yaml
with an outdated one
Also can you give this PR a more explicit name? I'm glad you're solving issues but that's not an ideal name haha
I know this is WIP but be careful: you have a weird
.bak
file which should not be committed and not pushed + you are overwriting the currentdefaults.yaml
with an outdated one
Understood. That's why I decided to leave these 2 most important files until I get up (with a clear mind) haha.
Also can you give this PR a more explicit name? I'm glad you're solving issues but that's not an ideal name haha
Got it!
I didn't integrate the ADVENTAdversarialLoss into the get_losses function because it's a little bit different. If you want me to put it into get_losses, just notify me.
Advent dicription: https://drive.google.com/file/d/13gbw5NNPANoWeFyIg7Pf8G4v_b8-53Az/view?usp=sharing
Google Docs
I will request a review when I am ready (finishing the confliction and resolving the conversation)