search

cmu-db / peloton

The Self-Driving Database Management System
http://pelotondb.io
Apache License 2.0
2.04k stars 623 forks source link

Travis failure due to tensorflow requirement not found #1448

Open lmwnshn opened 6 years ago

lmwnshn commented 6 years ago

On the first travis box, we fail the tensorflow test because pip cannot find tensorflow 1.4.0

Collecting tensorflow==1.4.0
[31m  Could not find a version that satisfies the requirement tensorflow==1.4.0 (from versions: )[0m
[31mNo matching distribution found for tensorflow==1.4.0

This affects recent PRs such as #1443 , #1446 .

mbutrovich commented 6 years ago

This seems to coincide with the Python 3.7 release on Homebrew a few days ago. Looks like tensorflow hasn't updated their compatibility list to include 3.7: https://github.com/tensorflow/tensorflow/issues/17022

Note the Programming Language on this page doesn't list 3.7 yet: https://pypi.org/project/tensorflow/

I suspect it'll be resolved soon enough no that 3.7 is the latest official release.

saatviks commented 6 years ago

Just to add to this, Also tried pip3 install --upgrade https://storage.googleapis.com/tensorflow/mac/${TF_TYPE}/tensorflow-${TF_VERSION}-py3-none-any.whl which is meant to be for all python versions, but isnt compatible with python3.7 currently. This is detailed in this issue.

tli2 commented 6 years ago

I think it is still failing on Travis last I checked (Your PR)

Maybe we should disable this for now?

saatviks commented 6 years ago

Yep we should disable it. Thats why I closed my PR.

tli2 commented 6 years ago

Temporarily fixed due to #1450, we should re-enable dependency when possible.