Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

validate numpy1.12 compatibility changes #5527

Merged
merged 16 commits into from
Feb 13, 2017

Conversation

lamblin
Copy link
Member

@lamblin lamblin commented Feb 10, 2017

This is just a rebase of #5461 to see if the issue happens with the main cache.

@slefrancois
Copy link
Collaborator

jenkins test this

@lamblin
Copy link
Member Author

lamblin commented Feb 10, 2017

Splitting off test_abstract_conv.py in a new PART seems to have worked for Travis, and the tests pass.
I'm going to revert the changes to the numpy and scipy version so it can be merged.
@nouiz or @slefrancois could you please have a look at my commits?

@nouiz
Copy link
Member

nouiz commented Feb 10, 2017 via email

Copy link
Member

@nouiz nouiz left a comment

Choose a reason for hiding this comment

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

ok to merge when travis pass.

@lamblin
Copy link
Member Author

lamblin commented Feb 11, 2017

I think I finally got down to the cache issue.
I'm trying once again with numpy 1.12, and if it passes, I'll revert the changes to travis again.

@lamblin lamblin force-pushed the ccw_numpy_travis branch 2 times, most recently from 1d93772 to eca5cab Compare February 11, 2017 21:38
@lamblin
Copy link
Member Author

lamblin commented Feb 11, 2017

So everything works fine, as long as the non-conda packages for scipy and numpy are not installed in Travis' cache (in which case it makes linking with blas fail), which I did by temporarily removing $HOME/miniconda2 from Travis' cache (but still keeping $HOME/.theano in there).

I'm confident everything is fine now, so I'm restoring .travis.yml and clearing the cache for that PR.
If someone can do a final review of 56a71e7, that would be great.

@lamblin
Copy link
Member Author

lamblin commented Feb 11, 2017

Actually, Jenkins was complaining as well.
I'll have a look at the GPU tests...

@lamblin
Copy link
Member Author

lamblin commented Feb 12, 2017

And this last one should fix the version for GpuCAReduceCPY (at least it did locally).
@nouiz or @abergeron could you confirm it is OK?

@lamblin lamblin merged commit e829ba4 into Theano:master Feb 13, 2017
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.

None yet

4 participants