imatge-upc / retrieval-2016-icmr

Bags of Local Convolutional Features for Scalable Instance Search (ICMR 2016)
https://imatge-upc.github.io/retrieval-2016-icmr/
111 stars 36 forks source link

Problem in A_feature_extraction.py #2

Closed willard-yuan closed 6 years ago

willard-yuan commented 7 years ago

The line L363 in A_feature_extraction.py is superfluous, and it need to comment to make it run successfully.

hurricane2018 commented 6 years ago

I modified the corresponding path in the settings.json file, then ran it. However, an error appeared: " NameError: name 'pathDB' is not defined " . My question is how did you define pathDB ? is it the prototxt file ?

Thank you very much!

evamohe commented 6 years ago

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, Eva

hurricane2018 commented 6 years ago

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 https://github.com/imatge-upc/retrieval-2016-icmr/issues/2.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/imatge-upc/retrieval-2016-icmr/issues/2#event-1475433034, or mute the thread https://github.com/notifications/unsubscribe-auth/AifZ-WSrw2ff0QXARSrhhYGbjkuO3RIzks5tU_V6gaJpZM4LO9M2 .