-
Notifications
You must be signed in to change notification settings - Fork 18.7k
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
Standard format for Caffe models to enable easy distribution with forthcoming "model zoo" #917
Conversation
Oh my, you deleted the braincells quote :) |
# 1. reference, and not casually committed | ||
# 2. custom, and live on their own unless they're deliberated contributed | ||
data/* | ||
examples/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm pretty sure examples should still be ignored. If you execute any ipynb it will change the contents since new matplotlib figures are generated but you hardly want to commit that.
4278286
to
c01f07a
Compare
@sergeyk rebased to fit paths to example standardization and took care of model extensions. Still need to fix model / path references in examples. |
I'm updating the Imagenet example now. |
@sergeyk awesome. Let me know if you need any help and I'll do a pass for merge. |
Okay I think this is done. Sadly, there is no commandline tool to download gists, and I'm not about to write github authentication code. We'll add that nicety later -- the upload works like clockwork now. |
Define standard format for Caffe models to open the "model zoo"
Thanks Sergey! This will be a good format for models coming out. A follow-up can add gist downloading. |
Define standard format for Caffe models to open the "model zoo"
Define standard format for Caffe models to open the "model zoo"
This PR will impose a certain structure on all example models we distribute, and add scripts to
Consult
docs/model_zoo.md
for the proposed changes.