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

hvplot/utils also requires pyproj #299

Closed
nbecker opened this issue Feb 11, 2019 · 3 comments · Fixed by #392
Closed

hvplot/utils also requires pyproj #299

nbecker opened this issue Feb 11, 2019 · 3 comments · Fixed by #392

Comments

@nbecker
Copy link

nbecker commented Feb 11, 2019

In util.py: process_crs, we see:

    try:
        import cartopy.crs as ccrs
        import geoviews as gv # noqa
        import pyproj
    except:
        raise ImportError('Geographic projection support requires GeoViews and cartopy.')

But pyproj is not mentioned, and this is exactly what I was missing.

@philippjfr
Copy link
Member

Thanks for tracking that down, I had assumed pyproj was installed alongside cartopy. I'll push a fix to hvplot and then tag a micro release later today or tomorrow.

@KarenChen9999
Copy link

I ran into the same issues,

/apps/opt/anaconda3/envs/geo_env/lib/python3.8/site-packages/hvplot/util.py in process_crs(crs)
    182         import pyproj
    183     except:
--> 184         raise ImportError('Geographic projection support requires GeoViews and cartopy.')
    185 
    186     if crs is None:

ImportError: Geographic projection support requires GeoViews and cartopy.

my package versions are:
geopandas 0.8.2 pyhd8ed1ab_0 conda-forge
geoviews 1.8.1 pypi_0 pypi
hvplot 0.7.0 pyhd3deb0d_0 conda-forge

@jbednar
Copy link
Member

jbednar commented Feb 15, 2021

@philippjfr , it looks like you made a change in how pyproj is handled in process_crs in geoviews/util.py, but (despite the repo this issue is filed on) the issue reported here was about process_crs in hvplot/util.py, not GeoViews. The hvPlot version still tries to import pyproj as it did before, and so hvPlot users with geoviews but without pyproj would presumably still see this message, which it sounds like is what @KarenChen9999 is reporting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants