Skip to content

Working notebook#3

Open
Yichuans wants to merge 7 commits intomasterfrom
develop
Open

Working notebook#3
Yichuans wants to merge 7 commits intomasterfrom
develop

Conversation

@Yichuans
Copy link
Owner

Description

  1. Added minimal python installation in the README
  2. Working notebook that includes MODIS data manipulation on GEE, visualisation (+legend), PPAPI to derive GeoJSON, zonal statistics and clipping (cartographic effect)

Improvement required

  1. Need refactoring and relocation to utility.py for easy importing and for migrating to Flask app
  2. Coherent narrative and improvement in the notebook text

@Yichuans Yichuans requested review from Levia and boc84 February 21, 2019 16:57
`docker run --name "gee" -it -p "127.0.0.1:8888:8888" -v ./workspace:/app gee`

```bash
docker run --name "gee" -it -p "127.0.0.1:8888:8888" -v ./workspace:/app gee
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Yichuans Can we please add some more info on what those parameters are?
I was trying to run it from the main app folder, but even using -v ./ or -v . was returning an error. While running the correct command from outside the main folder and pointing to that from -v has worked fine.

Moreover, there might be some instances of "gee" running, so there are some commands we could use to manage that:

  • docker stop gee where gee is the name of the container
  • docker rm gee which removes the container once it has been stopped
  • docker start gee which runs an existing container named gee which has been previously stopped

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

Successfully merging this pull request may close these issues.

2 participants