Disable TensorFlow tests to allow Travis macOS builds to succeed.

cmu-db / peloton

The Self-Driving Database Management System

http://pelotondb.io

Apache License 2.0

2.03k stars 623 forks source link

Disable TensorFlow tests to allow Travis macOS builds to succeed. #1449

Closed mbutrovich closed 6 years ago

mbutrovich commented 6 years ago

Workaround for #1448. This change should be reverted once the tensorflow package is updated with Python 3.7 compatibility.

saatviks commented 6 years ago

Thanks for pointing out the problem @mbutrovich An alternative as described here would be to install directly as pip3 install https://storage.googleapis.com/tensorflow/mac/cpu/tensorflow-1.4.0-py3-none-any.whl.

coveralls commented 6 years ago

Coverage decreased (-0.7%) to 76.467% when pulling fbb86f94da2d2e381600d7dbf4e1edd74cfa8cf2 on mbutrovich:disable_tf_test into 6cc5559477a3362a7bd7d56a0117ed05678e8e3d on cmu-db:master.

saatviks commented 6 years ago

Seems like the solution in #1450 also doesnt work as detailed in this issue here. Best to go ahead with this fix of disabling tensorflow tests. I'm closing mine. Also notifying @malin1993ml that we are doing this.