openforis / sepal

Geographical Data Processing in the Cloud
https://sepal.io/
MIT License
211 stars 46 forks source link

Test sepal_ui tools with latest sepal environment #302

Open dfguerrerom opened 1 year ago

dfguerrerom commented 1 year ago

With the latest sepal Ubuntu release, we'll have to make sure all our apps are working properly in the test server. I'm creating this issue to track the progress of this process, and it's open to anyone who wants to help on the testing.

The steps /objectives are:

SEPAL_UI

- `git clone git@github.com:12rambau/sepal_ui.git`
- `cd sepal_ui`
- `git checkout module_venv`  # this branch contains the module_venv bin file to mimic the way as sepal does the venvs
- `pip install -e . --config-settings editable_mode=strict`  **

** I'm still not knowing why it doesn't work with only pip install -e . as on my local installation... there must be something that doesn't allow this behavior... after losing too many time I could make it work with the extra args, but not the "compat" previous one... It happens that even though the editable .pth is located in .local/lib/python3.10/site-packages it will still preferring the not editable... I think this deserves to open an issue in setuptools because I couldn't go to the root of the problem.

Repository to test

edit requirements.txt file

Create local module virtual environment and test the app

12rambau commented 1 year ago

At this point don't you think it would make more sense to sit 2-3 days and set up some tests in the Jenkins pipeline ?

dfguerrerom commented 1 year ago

At this point don't you think it would make more sense to sit 2-3 days and set up some tests in the Jenkins pipeline ?

We definitely have to create some type of test to avoid doing this in the future, but I think the situation is a bit complex now because:

dfguerrerom commented 1 year ago

Another issue, some modules required rasterio==1.1.5, that version is incompatible with GDAL==3.6.4, meaning that we have to manually change to a one that works and check that everything works.

dfguerrerom commented 1 year ago

@cdanielw, @lpaolini (and just for curiosity @12rambau ) We should find a way to make the apps fully isolated in their own environment, using their own C Libraries, so we won't end in situations like this.