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

[JOSE] pypi installation #14

Closed
BenjMy opened this issue Dec 19, 2022 · 7 comments
Closed

[JOSE] pypi installation #14

BenjMy opened this issue Dec 19, 2022 · 7 comments

Comments

@BenjMy
Copy link

BenjMy commented Dec 19, 2022

xref openjournals/jose-reviews#185

The required dependencies should be installed automatically when you install GemGIS using pip.

pip install gemgis runs well but led to missing dependencies after importing GemGIS (matplolib, geopandas,...) .

You would probably need to revise your setup.py to add the necessary requirements.

@BenjMy BenjMy changed the title pypi installation [JOSE] pypi installation Dec 19, 2022
@AlexanderJuestel
Copy link
Collaborator

Dear @BenjMy,

thanks for this issue. There is an installation page on how to install GemGIS: https://gemgis.readthedocs.io/en/latest/getting_started/installation.html

I will certainly have to revise this one now that major packages have been updated (e.g. Shapely). We decided on having this step-by-step installation to make sure that every package is properly installed rather than installing everything at once and get problematic errors. GemGIS relies on packages that depend in return on GDAL (geopandas and rasterio) or VTK (pyvista) or Theano (gempy), which can be tricky to install within themselves.

What is your take as a reviewer on that? I would like to keep the installation of GemGIS simple if all major dependencies are properly installed correctly.

Cheers Alex

@AlexanderJuestel
Copy link
Collaborator

Dear @BenjMy,

in combination with your issue here, I will have to refactor some of the code of GemGIS as the pygeos package was merged with Shapely recently. This will make it hopefully a little easier to install GemGIS in the future if one dependency is gone.

@BenjMy
Copy link
Author

BenjMy commented Jan 11, 2023

Hello Alex,

Thanks for the details.

I understand your point and I think everyone experienced with python and pip would understand. But naively if I use pip install gemgis, I expect to have all the required dependencies installed... Hope this would be possible with the new version and the shapely integration.
I keep an eye on the Gemgis repo and will do a final check for the next round of the paper review.

Thanks.

@AlexanderJuestel
Copy link
Collaborator

Hey,

I can certainly try to get all the dependencies installed via pip for the GemGIS repo!

@AlexanderJuestel
Copy link
Collaborator

Hello @BenjMy,

after some consideration, I think we should not try to install GemGIS entirely via pip. If you look at the installation instructions for geopandas for using pip, no especially new user would want to "compile and install th[e] C dependencies [of the geopandas dependencies] manually". If you have the most important dependencies installed via conda (geopandas, pyvista, rasterio), then the installation of gemgis via pip is a charm. And we have the installation instructions for gemgis which are easy to follow.

image

@BenjMy
Copy link
Author

BenjMy commented Feb 17, 2023

Hi @AlexanderJuestel ,

Thanks for your effort in trying to dig out some possible solutions. I understand your concern about a full pip installation. As soon as the instructions and steps are super clear then I don't think it is an issue anymore (feel free to close).
Thanks.

@AlexanderJuestel
Copy link
Collaborator

The instructions were reworked and should be clear now, closing the issue

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

No branches or pull requests

2 participants