Skip to content
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

accuracy problem #8

Closed
Mrzhangxiaohua opened this issue Nov 25, 2019 · 6 comments
Closed

accuracy problem #8

Mrzhangxiaohua opened this issue Nov 25, 2019 · 6 comments

Comments

@Mrzhangxiaohua
Copy link

I am interested in your work and i had run your code of cmd successfully but i still have two problems.

  1. the total accuracy on office31, am->wc, wc ->am, etc are lower than your paper results.
    my: acc-tst=0.5597484278229048; acc-tst-dr=0.6503144648090099
    yours:acc-tst = .676 +-006 ; acc-tst-dr= .770 +- 006
  2. i use tensorflow==0.12.1 and other same as you, how can i put it on GPU
    (i use keras function: from keras.utils.multi_gpu_utils import multi_gpu_model,
    with no err but run it i got ModuleNotFoundError: No module named 'keras.utils.multi_gpu_utils')
@wzell
Copy link
Owner

wzell commented Nov 27, 2019

Hallo Mrzhangxiaohua,

nice to see Your interest in our project!

You are using a different framework than we do. Please install the libraries as described in the requirements and requirements.txt.

Since the domain adaptation works (accuracy improvement of 10% is the same as in our experiments), I assume that the base model VGG16 is somehow different than in our implementations.

The right framework should change this problems. However, if you want to proceed with your framework I suggest to contact the respective pages.

I allow myself to close this issue, as it is clearly related to some untested differences in the software other than specified in the requirements.

@wzell wzell closed this as completed Nov 27, 2019
@Mrzhangxiaohua
Copy link
Author

thanks for your replying me! flower
I use the same framwork and code, but keras 1.1 require tf as base,so i use tensorflow==0.12.1
and the data and vgg16 weight from your readme link.
and wc->dslr, dslr->wc is right(same as your paper), others are lower, so i don't known where i did wrong

@wzell
Copy link
Owner

wzell commented Nov 27, 2019

Our implementation is based on theano not tensorflow. Keras does NOT require tensorflow but it can also be based on Theano.

@Mrzhangxiaohua
Copy link
Author

ok, i will have a try. Thank you very much

@wzell
Copy link
Owner

wzell commented Nov 27, 2019

Maybe this helps you also: There is an extended version of our work available in https://github.com/wzell/mann

There is an illustrative example that shows how to update the code to newer versions of keras and tensorflow.

@Mrzhangxiaohua
Copy link
Author

thank you very much

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants