Skip to content
This repository has been archived by the owner on Jan 3, 2018. It is now read-only.

404 error for new Python lessons #282

Closed
rgaiacs opened this issue Feb 3, 2014 · 4 comments
Closed

404 error for new Python lessons #282

rgaiacs opened this issue Feb 3, 2014 · 4 comments

Comments

@rgaiacs
Copy link

rgaiacs commented Feb 3, 2014

Problem

If you follow README.md for setup bootcamp using the new Python lessons the Python lessons won't be available.

Steps to reproduce

Note: the 1st step should take some time. You can start with the 2nd step and use http://geocarpentry.github.io/2014-01-30-mit/python/novice/ instead of http://{your-github-username}.github.io/YYYY-MM-DD-site/python/novice.

  1. Follow the instructions in README.
  2. Open http://{your-github-username}.github.io/YYYY-MM-DD-site/python/novice with your internet browser.
  3. Click in any links of "Topics" section.

Actual results

404 error for http://geocarpentry.github.io/2014-01-30-mit/python/novice/01-numpy.html

Expected results

Got the lesson.

Solution

Need to add Markdown version of the IPython Notebook into the repository ONLY WHEN DEPLOYING THE BOOTCAMP.

Note I already start to write the PR for this but want to use the changes from #252.

Meta I want to add a new command in Makefile to make easy this part of setup. @gvwilson any suggest for command's name?

@wking
Copy link
Contributor

wking commented Feb 3, 2014

On Sun, Feb 02, 2014 at 06:53:23PM -0800, r-gaia-cs wrote:

Need to add Markdown version of the IPython Notebook into the
repository ONLY WHEN DEPLOYING THE BOOTCAMP.

Did you mean “add an HTML version”? That makes more sense to me, and…

Note I already start to write the PR for this but want to use
the changes from #252.

#252 is about rendering IPyNbs to HTML.

Meta I want to add a new command in Makefile to make easy this
part of setup. @gvwilson any suggest for command's name?

Doesn't 'make check' already do this? Or do you want to give
NOTEBOOK_DST it's own target?

@ahmadia
Copy link
Contributor

ahmadia commented Feb 3, 2014

@r-gaia-cs - Thanks for the report. If I understand the situation correctly:

In the master repository, we keep .ipynb files, but not the corresponding .html files that are compiled using nbconvert.

When we use make check, the instructor gets a local copy of the repository, including static views of the notebooks. However, the static views of the notebooks are not committed in the gh-pages branch, and therefore not visible.

What do you think about just using nbviewer to dynamically render the notebooks? You can see the complete links from the MIT agenda on the front page for how we did this, but here's an example: http://nbviewer.ipython.org/github/geocarpentry/2014-01-30-mit/blob/gh-pages/python/intermediate/01-intro-python.ipynb

@rgaiacs
Copy link
Author

rgaiacs commented Feb 3, 2014

Need to add Markdown version of the IPython Notebook into the
repository ONLY WHEN DEPLOYING THE BOOTCAMP.

Did you mean “add an HTML version”? That makes more sense to me, and…

Note I already start to write the PR for this but want to use
the changes from #252.

#252 is about rendering IPyNbs to HTML.

#252 use Markdown as intermediate format and leave jekyll build the HTML.

Meta I want to add a new command in Makefile to make easy this
part of setup. @gvwilson any suggest for command's name?

Doesn't 'make check' already do this? Or do you want to give
NOTEBOOK_DST it's own target?

make check build the static view. I want one command that build the
intermediate Markdown file and commit it leaving GitHub Pages build the HTML.

@ahmadia

@r-gaia-cs - Thanks for the report. If I understand the situation correctly:

In the master repository, we keep .ipynb files, but not the corresponding
.html files that are compiled using nbconvert.

When we use make check, the instructor gets a local copy of the repository,
including static views of the notebooks. However, the static views of the
notebooks are not committed in the gh-pages branch, and therefore not
visible.

You are correct.

What do you think about just using nbviewer to dynamically render the
notebooks? You can see the complete links from the MIT agenda on the front
page for how we did this, but here's an example:
http://nbviewer.ipython.org/github/geocarpentry/2014-01-30-mit/blob/gh-pages/python/intermediate/01-intro-python.ipynb

I prefer have a uniform view for the lessons (#252) but I'm OK if we agree using
nbviewer (in this case I'm +1 in change the links in python/novice/index.html
and others to point at nbviewer).

@rgaiacs
Copy link
Author

rgaiacs commented Feb 23, 2014

For bootcamps using the v5 lessons students should use the content at http://software-carpentry.org/v5/ as explained in @gvwilson video tutorial How setting up a GitHub
repository for a bootcamp
.

@rgaiacs rgaiacs closed this as completed Feb 23, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants