Open-EO / openeo-udf

The OpenEO UDF Python reference implementation and interface description
https://open-eo.github.io/openeo-udf/
Apache License 2.0
6 stars 1 forks source link

Install: Cannot find tensorflow<=2.1.0 #35

Open ahalota opened 3 years ago

ahalota commented 3 years ago

Following the instructions on the readme. Because fastapi branch was not available, I ignored that and continued with the instructions on main branch.

With the line for pip3 install -r requirements.txt I am getting this error message:


ERROR: Could not find a version that satisfies the requirement tensorflow<=2.1.0
ERROR: No matching distribution found for tensorflow<=2.1.0

I'll try and ignore this and continue with tensorfow 2.4

ahalota commented 3 years ago

I don't think ignoring that version requirement was a good solution. As I continue trying to install it seems the script is trying to download many versions of tensor flow. (2.2,2.3,2.4...etc).

Similarly when it reaches the stage of looking for scipy, it downloaded very many of them:

 Using cached scipy-0.17.1.zip (13.8 MB)
  Using cached scipy-0.17.0.zip (13.8 MB)
  Using cached scipy-0.16.1.zip (13.6 MB)
  Using cached scipy-0.16.0.zip (19.3 MB)
  Using cached scipy-0.15.1.zip (12.7 MB)
  Using cached scipy-0.15.0.zip (12.7 MB)
  Using cached scipy-0.14.1.zip (12.2 MB)
  Using cached scipy-0.14.0.zip (11.5 MB)
  Using cached scipy-0.13.3.zip (12.1 MB)
  Using cached scipy-0.13.2.zip (11.4 MB)
  Using cached scipy-0.13.1.zip (11.3 MB)
  Using cached scipy-0.13.0.zip (11.3 MB)
  Using cached scipy-0.12.1.zip (10.2 MB)
  Using cached scipy-0.12.0.zip (10.2 MB

I'm not sure if this is correct?

Either way I have not been able to succeed on install so far. I am trying to install openeo-udf since it seems to be needed for actinia grass server (a dependency for one of the additional modules listed on their install instructions). Are there up to date install instructions available?

jdries commented 3 years ago

My advice would be to remove the tensorflow dependency. Normally, it should only be there for users that actually want to use Tensorflow inside a UDF. It should not be needed strictly by the package. I believe I even made some imports of tensorflow optional in the past, only can't guarantee that none are left.