google-research / exoplanet-ml

Machine learning models and utilities for exoplanet science.
Apache License 2.0
296 stars 117 forks source link

Tensorflow version issues #9

Open ritwik12 opened 4 years ago

ritwik12 commented 4 years ago

While running test cases using Bazel, I found that using default TensorFlow installation i.e. tensorflow==2.0 a lot of test cases will fail. Few functionalities used in this project are deprecated in TF 2.0.

Working version of tensorflow dependencies are:

tensorboard            1.15.0    
tensorflow             1.15.0    
tensorflow-estimator   1.15.1    
tensorflow-probability 0.8.0  

Also, as per the conclusions from this issue we need to update tensorflow version in README.md and requirements.txt in #8

PravNJ commented 4 years ago

Thans. Just came across the same issue, going to try these versions and report back.

wiggs73 commented 4 years ago

@ritwik12 I have installed the same versions of the dependencies and have tried v3.1 of bazel as well as v2.0. But I still have build issues. The first issue is: ERROR: C:/projects/exoplanet-ml/exoplanet-ml/light_curve/BUILD:74:1: name 'py_proto_library' is not defined (did you mean 'cc_proto_library'?) Any thoughts on what to try next?

ritwik12 commented 4 years ago

@wiggs73 You need to follow #6 for this. Also, see discussion over there and you'll get your issues solved.