gee-community / ee-jupyter-contrib

Using Jupyter as an Earth Engine IDE - Community Contributions
MIT License
73 stars 42 forks source link

Add instructions for reviewing notebook ipynb files #11

Open tylere opened 6 years ago

tylere commented 6 years ago

Instructions for viewing ipynb files are needed. Instructions for diff and merging ipynb files are needed.

tylere commented 6 years ago

One approach (for new ipynb files):

  1. Use git to clone the repo locally.
  2. Switch to the branch with the PR changes.
  3. Start up Jupyter/JuypterLab to view the current state of the notebook doc.
  4. Run nbdime (nbdiff-web) to view changes to the notebook doc.
  5. Comment on the Github PR.
tylere commented 6 years ago

The ee-jupyter-contrib/docker/jupyter-scipy-notebook-ee/ Dockerfile now installs nbdime, which is useful for reviewing changes to notebooks.