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

train #28

Open
tkone2018 opened this issue Dec 25, 2018 · 9 comments
Open

train #28

tkone2018 opened this issue Dec 25, 2018 · 9 comments
Labels
help wanted Extra attention is needed

Comments

@tkone2018
Copy link

hi, i have a problem when i run the train.py
error: argyment --training-dataset/-d: invalid choice: 'retrieval-SfM-120k',
can you give me some advices?thanks

@Salehbigdeli
Copy link

The default dataset for training is retrieval-SfM-120k it's not necessary to set it for train. Use newest version of code.

@filipradenovic
Copy link
Owner

Have you tried running the train example from README training section? I.e., have you tried running the following line:

python3 -m cirtorch.examples.train YOUR_EXPORT_DIR --gpu-id '0' --training-dataset 'retrieval-SfM-120k' 
            --test-datasets 'roxford5k,rparis6k' --arch 'resnet101' --pool 'gem' --loss 'contrastive' 
            --loss-margin 0.85 --optimizer 'adam' --lr 1e-6 --neg-num 5 --query-size=2000 
            --pool-size=22000 --batch-size 5 --image-size 362

@filipradenovic filipradenovic added the help wanted Extra attention is needed label Jan 12, 2019
@Garamda
Copy link

Garamda commented Jan 14, 2019

I eliminated quotation marks ex) 'retrieval-SfM-120k' -> retrieval-SfM-120k and it worked. I don't know why the quotation marks need to be removed though. Other quotation marks in the command should be removed in the same way.

p.s. I'm using Window 10, not Linux.

@filipradenovic
Copy link
Owner

@Garamda

p.s. I'm using Window 10, not Linux.

This might be the reason, we have only tested it on Linux, so far. It is good to know the code runs on Windows as well.

@Garamda
Copy link

Garamda commented Jan 14, 2019

@filipradenovic

However, because of Linux commands like wget or rm, I couldn't go further. I think I should use Linux.

@filipradenovic
Copy link
Owner

@Garamda

If the problem is only in the download script, then this can be solved by manually downloading necessary files from cnnimageretrieval website. It should be clear from the download script how the data should be arranged (folder structure). I don't know if it is too much work to do it manually though, but it is only done once, so it might be worth it.

Hope this helps anyhow.

@Garamda
Copy link

Garamda commented Jan 14, 2019

@filipradenovic

That is great idea. Thank you :)

@filipradenovic
Copy link
Owner

@Garamda if you do manage to run it with Windows, can you post your findings here? It might be very useful for many other Windows users, and we would be very grateful. If it is easier for you, you can even open a new issue named 'Running the toolbox on Windows'. Thank you.

@Garamda
Copy link

Garamda commented Jan 14, 2019

@filipradenovic

Alright. If I can, I ll do it :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants