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

Model training Loss graph #3

Open
Ugness opened this issue Oct 4, 2018 · 6 comments
Open

Model training Loss graph #3

Ugness opened this issue Oct 4, 2018 · 6 comments

Comments

@Ugness
Copy link

Ugness commented Oct 4, 2018

newplot

Plot.ly Link: https://plot.ly/~Ugness/1/

I am training the model with ILSVRC2012 training set and same options as your implementation and my loss graph looks like above.
I am afraid that my model's loss reduces correctly. Can you check this loss graph or share your loss graph?
Thanks.

@richzhang
Copy link
Owner

Thanks for the message. I'll look into it in the next week or so.

@richzhang
Copy link
Owner

My apologies for the delay. I'm caught up with a few things and will have to look into this more after CVPR deadline (mid-Nov).

@Ugness
Copy link
Author

Ugness commented Nov 7, 2018

Thanks.

@Samir55
Copy link

Samir55 commented Dec 25, 2018

Hello author,
I have the same problem above, what is the solution?
Thanks in advance.

@richzhang
Copy link
Owner

I believe I did not do the initialization properly. I am looking into it. Thanks for your patience!

@richzhang
Copy link
Owner

richzhang commented Jan 5, 2019

The problem was with the initialization. In the original paper, I used k-means initialization, but didn't have the code for pytorch.

I just pushed a change. If you rerun python make_ilsvrc_dataset.py, it will make a directory dataset/ilsvrc2012/train_small/, which contains 10 imagenet classes. Then rerun bash scripts/train_siggraph.sh again. It will first run 100 epochs on this small training set as "initialization", before training on the full training set.

It does still train quite slowly. Better networks/loss functions would be a good update to the project.

Thanks for your patience.

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

3 participants