Skip to content

removing notebooks in docs, updating instructions #234

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Mar 18, 2014
Merged

removing notebooks in docs, updating instructions #234

merged 2 commits into from
Mar 18, 2014

Conversation

sergeyk
Copy link
Contributor

@sergeyk sergeyk commented Mar 18, 2014

  • Added instructions for downloading pretrained models.
  • Got rid of notebooks in docs/: now directly link to nbviewer. Keeping things DRY!
  • Updated example notebooks with correct links to instructions
  • Refreshed index page

shelhamer
shelhamer reviewed Mar 18, 2014
View reviewed changes
docs/index.md Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"academic / non-commercial use"

@sergeyk
Copy link
Contributor Author

sergeyk commented Mar 18, 2014

Incorporated your suggestions.

sguada
sguada reviewed Mar 18, 2014
View reviewed changes
docs/getting_pretrained_models.md
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was trained on ILSVRC-2012

@shelhamer
Copy link
Member

Nice re-arrangement Sergey. One thing to note with the ipynbs in nbviewer now is that they need to be downloaded to examples/ for them to work with the given paths.

@sergeyk
Copy link
Contributor Author

sergeyk commented Mar 18, 2014

They are in examples/ already, no?

On Tue, Mar 18, 2014 at 12:24 PM, Evan Shelhamer
[email protected]:

Nice re-arrangement Sergey. One thing to note with the ipynbs in nbviewer
now is that they need to be downloaded to examples/ for them to work with
the given paths.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/234#issuecomment-37976633
.

shelhamer added a commit that referenced this pull request Mar 18, 2014
@shelhamer
Merge pull request #234 from sergeyk/updating_docs
8ee753d 
removing notebooks in docs, updating instructions
@shelhamer shelhamer merged commit 8ee753d into BVLC:dev Mar 18, 2014
@shelhamer
Copy link
Member

@sguada let's each do a revision pass and push to dev. Then I'll update #231 and do the release.

@shelhamer
Copy link
Member

@sergeyk right, sorry I hallucinated their removal from examples as well as docs.

mitmul pushed a commit to mitmul/caffe that referenced this pull request Sep 30, 2014
@shelhamer
Merge pull request BVLC#234 from sergeyk/updating_docs
97027c0 
removing notebooks in docs, updating instructions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@shelhamer shelhamer

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sergeyk @shelhamer @sguada @kloudkl