geopython-workshop icon indicating copy to clipboard operation
geopython-workshop copied to clipboard

`rasterio` dependency is missing in requirements.txt file

Open krishnaglodha opened this issue 1 year ago • 3 comments

I am trying to run jupyter notebook locally, but in geopython-workshop/workshop/jupyter/requirements.txt rasterio is missing

krishnaglodha avatar Jun 22 '23 03:06 krishnaglodha

I see, but I am hesitant to change geopython-workshop/workshop/jupyter/requirements.txt at this point in time. This file is also used in the Dockerfile and -build where rasterio is installed as a Debian (python3-rasterio) package. There could be potential dependency conflicts, resulting in a failing Notebook. Everything has to be tested again, begin to end. We have a stable situation now. Maybe rasterio was even taken out of requirements.txt for that reason. That file is tuned for the Ubuntu env and .deb installs in the Dockerfile.

At the workshop we will use Docker. Custom install is at own risk. Will need its own requirements.txt , install guide, and base deps like GDAL and Proj. Maybe Conda Forge works better for that case. That would be a useful addition.

justb4 avatar Jun 22 '23 08:06 justb4

@justb4 got it! Thanks :D

krishnaglodha avatar Jun 22 '23 09:06 krishnaglodha

I agree it is a bit too late to try this kind of change. The notebook in Section 5 is working fine as it is.

We can give it a try later during the code sprint.

ldesousa avatar Jun 22 '23 14:06 ldesousa