Skip to content

Add TensorFlow workaround for our Jenkins server - #357

Merged
mihaic merged 1 commit into
brainiak:masterfrom
mihaic:add-conda-override
Apr 16, 2018
Merged

Add TensorFlow workaround for our Jenkins server#357
mihaic merged 1 commit into
brainiak:masterfrom
mihaic:add-conda-override

Conversation

@mihaic

@mihaic mihaic commented Apr 9, 2018

Copy link
Copy Markdown
Member

We want to be able to use TensorFlow as a dependency. TensorFlow
currently requires glibc >= 2.17 (not formally documented, see
tensorflow/tensorflow#53 (comment)).
We are currently running Jenkins on CentOS 6, which has glibc < 2.17.
Installing TensorFlow with Conda also installs the required glibc.

We want to be able to use TensorFlow as a dependency. TensorFlow
currently requires glibc >= 2.17 (not formally documented, see
tensorflow/tensorflow#53 (comment)).
We are currently running Jenkins on CentOS 6, which has glibc < 2.17.
Installing TensorFlow with Conda also installs the required glibc.
@mihaic

mihaic commented Apr 9, 2018

Copy link
Copy Markdown
Member Author

This is how I tested on the Jenkins server:

conda create -n tmp python=3
source activate tmp
python3 -m pip install tensorflow
python3 -c "import tensorflow"  # fail
conda install --yes tensorflow
python -c "import tensorflow"  # success

@mshvartsman mshvartsman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@mshvartsman

Copy link
Copy Markdown
Contributor

Hi @mihaic, can this be merged, or do we need someone else to review / approve?

@mihaic
mihaic merged commit e503afe into brainiak:master Apr 16, 2018
@mihaic
mihaic deleted the add-conda-override branch April 16, 2018 16:55
@mihaic

mihaic commented Apr 16, 2018

Copy link
Copy Markdown
Member Author

Thanks for reviewing, @mshvartsman. Merged.

mihaic added a commit to mihaic/brainiak that referenced this pull request Apr 19, 2018
PR brainiak#357 did not fix the problem.

This time I verified by applying this commit on the branch in PR brainiak#341.
mihaic added a commit that referenced this pull request Apr 30, 2018
PR #357 did not fix the problem.

This time I verified by applying this commit on the branch in PR #341.
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

Successfully merging this pull request may close these issues.

2 participants