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

Use geopandas to plot all vectors #24

Closed
5 tasks done
juseg opened this issue Oct 10, 2022 · 0 comments
Closed
5 tasks done

Use geopandas to plot all vectors #24

juseg opened this issue Oct 10, 2022 · 0 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@juseg
Copy link
Owner

juseg commented Oct 10, 2022

I was looking for a way to combine shapefile geometries in a single artist, decided to give a quick try to GeoPandas and it blew my mind. Here is a quick speed test merging Ehlers et al. LGM global and alps paleoglaciers, removing duplicates, and plotting in Orthographic projection (as in the logo) with:

  • Cartopy BasicReader in 2m41s,
  • Cartopy FionaReader in 2m45s,
  • GeoPandas read_file in 5s!

Besides the increase in speed, GeoPandas will simplify hyoga internals and provide users with more flexible objects for further manipulation or plotting styles. However this will also introduces changes in API. To avoid future breaking change or deprecation warnings, I think I want to get this right before v0.2.

  • Use GeoPandas for Natural Earth.
  • Use GeoPandas for Paleoglaciers.
  • Add hyoga.open.naturalearth?
  • Add hyoga.open.paleoglaciers?
  • Update setup.cfg, doc/environment.yml.
@juseg juseg added the enhancement New feature or request label Oct 10, 2022
@juseg juseg added this to the v0.2.0 milestone Oct 10, 2022
@juseg juseg self-assigned this Oct 10, 2022
juseg added a commit that referenced this issue Oct 16, 2022
Apparently streamplot needs all variables (including coords) as numpy arrays,
not xarray data arrays. Why this issue is only showing up on non-cartopy axes
(and thus related to #24) is complete mistery to me.
juseg added a commit that referenced this issue Oct 16, 2022
Crazy how simple things become. I may need to merge pages, again rethink the
documentation outline. The beauty of geopandas is that we simply don't need to
import it or even think about it. Object-oriented.
@juseg juseg closed this as completed in 749d6ea Oct 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant