-
Notifications
You must be signed in to change notification settings - Fork 107
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
So is this code actually working? #5
Comments
@mario98 |
https://github.com/YeolJ00/mnist-svhn-transfer |
@YeolJ00 Hi there, would you please point out the problem in this code and the change you made? |
@EvaFlower Putting aside the deprecated functions, |
@YeolJ00 Yeah, you are right. But it did not raise exceptional information when running the code. So maybe it has no influence. |
@EvaFlower What do you mean by exceptional information? In what sense do you mean by it has no influence? |
@YeolJ00 I mean that I can run the code successfully without changing the transform for MNIST. So maybe here it just used the transform in the first channel and ignored others. |
@EvaFlower Well good for you. I wasn't able to run the code without changing that part. I wonder what the difference was. |
I get a reasonable result, when set "--use_reconst_loss True" and "--use_labels True"。 the MINST's generator result is better than svhn's |
So is this code actually working? Readme says "(should be re-uploaded)", What does that mean? Some people here report that this code is just memorizing training examples but does not generalize. Is this true?
Even worse than having no code is having code that is pretending to work but will just waste your time and then you have to implement it from scratch anyway...
The text was updated successfully, but these errors were encountered: