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

Install optional geopandas dependency on try-gmt? #24

Closed
weiji14 opened this issue Jul 19, 2021 · 0 comments · Fixed by #25
Closed

Install optional geopandas dependency on try-gmt? #24

weiji14 opened this issue Jul 19, 2021 · 0 comments · Fixed by #25
Labels
question Further information is requested

Comments

@weiji14
Copy link
Member

weiji14 commented Jul 19, 2021

Should we install geopandas (an optional dependency of PyGMT) on try-gmt? Just asking because a new contributor trying to work on GenericMappingTools/pygmt#1374 is having trouble with setting up PyGMT and Geopandas locally, and it would be nice to have a virtual cloud environment with both these dependencies installed already.

Disadvantage is that the build and start-time for the Binder environment will take longer, because geopandas brings in some heavy dependencies like GEOS and PROJ (see https://geopandas.org/getting_started/install.html#dependencies).

Thoughts? Upvote 👍 to go for it, downvote 👎 for sticking to the status quo.

@weiji14 weiji14 added the question Further information is requested label Jul 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant