AIX360 icon indicating copy to clipboard operation
AIX360 copied to clipboard

Upgrade the tensorflow version in setup.py to support higher versions of Python

Open elukey opened this issue 1 year ago • 4 comments

Hi folks!

The setup.py references tensorflow==1.14, that IIUC supports only Python 3.7, going to be EOL very soon. Would it be possible to bump the dependency up to support at least python 3.8+?

elukey avatar Apr 03 '23 13:04 elukey

https://github.com/Trusted-AI/AIX360/pull/150 https://github.com/Trusted-AI/AIX360/pull/149

elukey avatar Apr 08 '23 06:04 elukey

@vijay-arya @michaelhind @animeshsingh @krvarshney Hi! Sorry for the extra ping but the above PRs would probably help a lot of people since Python 3.7 is going EOL soon and a lot of projects now require 3.8+. If you have time could you please check if the tensorflow dep can be updated? Thanks in advance :)

elukey avatar Apr 08 '23 06:04 elukey

@elukey You can install each algorithm separately with different dependencies now, so tf version should not be an issue. Most algorithms are compatible with higher versions of python. Please see new clone + setup instructions here

vijay-arya avatar Jun 25 '23 07:06 vijay-arya

Hello, I was wondering why the shap algorithm requires version 1.14.0 of tensorflow: https://github.com/Trusted-AI/AIX360/blob/75eb9fbe44f18304d50a8ec77bcda1f132ecd172/setup.py#L75 when this requirement does not seem to be enforced by the shap package? https://github.com/shap/shap/blob/b92aef0b6a3264574db31556ca26236b5994a387/pyproject.toml#L72

Thank you :)

danibene avatar Jul 25 '23 01:07 danibene