Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

using EconML with TensorFlow-2.3.1 #352

Open
smoors opened this issue Dec 21, 2020 · 2 comments
Open

using EconML with TensorFlow-2.3.1 #352

smoors opened this issue Dec 21, 2020 · 2 comments

Comments

@smoors
Copy link

smoors commented Dec 21, 2020

the TensorFlow version requirements for EconML-0.8.1 are set to: tensorflow > 1.10, < 2.3

is this a hard requirement?
would it be ok to use TensorFlow-2.3.1 ?

@arose13
Copy link
Contributor

arose13 commented Mar 3, 2021

Tensorflows hard version requirements are creating substantially more problems than I think MSFT expected.

@kbattocchi
Copy link
Collaborator

kbattocchi commented Mar 3, 2021

We've made a change in version 0.9.0 to move the tensorflow requirement to an extra; tensorflow is only used by our DeepIV estimator, so now if you install just econml there is no tensorflow requirement at all (but you won't be able to use DeepIV), or you can install econml[tf] or econml[all], which will still require tensorflow > 1.10, < 2.3 and enable you to use DeepIV.

Relaxing the dependency is still something we'd like to achieve eventually, but there were breaking changes in keras and tensorflow that make it nontrivial. I'll look into whether a tensorflow < 2.4 requirement would be more achievable than removing the upper bound entirely, which was my previous intention.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants