Skip to content

dev: Remove superseded workaround from PR #408 - #410

Closed
mihaic wants to merge 1 commit into
brainiak:masterfrom
mihaic:remove-numpy-pin
Closed

dev: Remove superseded workaround from PR #408#410
mihaic wants to merge 1 commit into
brainiak:masterfrom
mihaic:remove-numpy-pin

Conversation

@mihaic

@mihaic mihaic commented Jan 22, 2019

Copy link
Copy Markdown
Member

New Theano release works with NumPy>=1.16.

New Theano release works with NumPy>=1.16.
@mihaic

mihaic commented Jan 22, 2019

Copy link
Copy Markdown
Member Author

Jenkins, retest this please.

2 similar comments
@mihaic

mihaic commented Jan 22, 2019

Copy link
Copy Markdown
Member Author

Jenkins, retest this please.

@mihaic

mihaic commented Jan 23, 2019

Copy link
Copy Markdown
Member Author

Jenkins, retest this please.

@mihaic

mihaic commented Jan 24, 2019

Copy link
Copy Markdown
Member Author

The error is caused by the different versions of NumPy available in PyPI and the default Conda repository. We introduced the potential for conflict in PRs #357 and #359 to be able to use TensorFlow on our CentOS 6 server for Jenkins (Metacortex).

For the moment, I will close this PR and thus keep NumPy pinned <1.16.

In the long term, it would be good to remove the workaround by switching the Jenkins server. I opened issue #411 to continue the conversation.

@mihaic mihaic closed this Jan 24, 2019
@mihaic
mihaic deleted the remove-numpy-pin branch January 24, 2019 00:09
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.

1 participant