-
Notifications
You must be signed in to change notification settings - Fork 36
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
Problem in A_feature_extraction.py #2
Comments
I modified the corresponding path in the settings.json file, then ran it. However, an error appeared: " NameError: name 'pathDB' is not defined " . Thank you very much! |
Hi @hurricane2018, If you are running the 'A_feature_extraction.py', pathDB should be automatically created by the 'LeveldbStorer' based on the 'featuresDB' and 'input_size' specified in your settings. Please, make sure you have specified those parameters in the settings file. I also encourage you to take a look to our recent paper "Saliency Weighted Convolutional Features for Instance Search" where you can find an updated (and probably cleaner/easy to use) version of the code. Best, |
Dear Eva:
Thank you for your reply. I have modified the values in
settings.json. However, the default value of pathDB is " pathDB='/media/eva/Eva
Data/icmr_results/trecvid_subset/features/vgg16/768_576/pool5_db ".
Actually, I don't know what is pool5_db. Is pool5_db a file folder ?
Thank you very much!
2018-02-15 4:14 GMT-05:00 Eva Mohedano <notifications@github.com>:
… Closed #2 <#2>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AifZ-WSrw2ff0QXARSrhhYGbjkuO3RIzks5tU_V6gaJpZM4LO9M2>
.
|
The line L363 in
A_feature_extraction.py
is superfluous, and it need to comment to make it run successfully.The text was updated successfully, but these errors were encountered: