Skip to content
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

Example interfacing GEE with contextily #147

Merged
merged 5 commits into from
May 13, 2020

Conversation

darribas
Copy link
Collaborator

@darribas darribas commented May 13, 2020

First stub, unfinished. Eventually, this PR will contain:

  • Notebook with example linking contextily to GEE (elevation model and Sentinel 2 Landsat 8)
  • Hook to display the notebook on the website

Also, with this, we'd be starting the series of interfacing contextily with other packages. My suggestion here is to use the name of the notebook friends_XXX.ipynb where XXX is the package we show how to interface. So, for example, we could also have friends_cenpy.ipynb, friends_osmnx.ipynb, etc. These should also display ideally under a horizontal bar on the side menu of the website (would this be possible @jorisvandenbossche?).

Will confirm when this can be reviewed/merged.

@darribas darribas self-assigned this May 13, 2020
@ljwolf
Copy link
Member

ljwolf commented May 13, 2020

We could throw a basemap on this?

@darribas
Copy link
Collaborator Author

OK I think the GEE is ready to go, at least for now. I've added it as a separate notebook but not separate (not sure how to do that)

@darribas darribas changed the title [WIP] Merge example interfacing GEE with contextily Merge example interfacing GEE with contextily May 13, 2020
@jorisvandenbossche jorisvandenbossche changed the title Merge example interfacing GEE with contextily Example interfacing GEE with contextily May 13, 2020
@jorisvandenbossche
Copy link
Member

To just add it to the sidebar on the doc site for now, you need to update:

@darribas
Copy link
Collaborator Author

Added! This is ready for review.

@darribas darribas merged commit cc97f2b into geopandas:master May 13, 2020
jpn-- added a commit to jpn--/contextily that referenced this pull request Jan 8, 2021
…iles

* commit '3c26d5a16a094e8d5ae05f8e5a41f05278816daf':
  DOC: update for latest pydata-sphinx-theme (fix sidebar + use CSS variables) (geopandas#168)
  RLS: v1.0.1
  Fix resetting of extent with local files (geopandas#155) (geopandas#156)
  Add random user ID for Nominatim and allow arbitrary geocoder (geopandas#164)
  Close and reopen memfile dataset before WarpedVRT (geopandas#165)
  Fix use of rasterio MemoryFile (geopandas#163)
  DOC: Use Cape Town consistently in description and variable names (geopandas#154)
  Add matplotlib Framework classifier (geopandas#152)
  Require Python 3.6 as minimum version (geopandas#150)
  Add earthengine to docker for binder
  Example interfacing GEE with contextily (geopandas#147)
  DOC: add notebook about working with local files (geopandas#139)

# Conflicts:
#	tests/test_ctx.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants