search

sepal-contrib / clip-time-series

Create a clip time series for a list of points
MIT License
1 stars 2 forks source link

no "refresh token" gee #98

Closed dfguerrerom closed 1 year ago

dfguerrerom commented 1 year ago

when venv is created, the earthengine-api line in requirements.txt is overwriting sepal gee-api installation, we should remove it

12rambau commented 1 year ago

you should instead leave "earthengine-api" alone without any pins. To make sure it continues to work elswhere. As it's already installed in SEPAL venv it will not change version

dfguerrerom commented 1 year ago

that's exactly how it is on release, and it triggers the error.

12rambau commented 1 year ago

was it updated since last week ? we changed the installation order of earthengine-api last week on prod (so anything that has not been updated since then should have badly designed venv)

dfguerrerom commented 1 year ago

ah okay... the latest venv update for this app was on May 4 at 10 am... by that moment I think SEPAL build was installing the app requirements before installing the forked version of earthengine-api... that's why you changed the order in https://github.com/openforis/sepal/commit/556d30b258fe27a2a47e55f7beace12c8432be4a. I will rebuild the venv

dfguerrerom commented 1 year ago

fixed with 38e348d35ad2909b0efbfb21e6ba8dc90ffe3fbd