Skip to content

BLD: Updading travis scipy package.#960

Merged
bblay merged 2 commits intoSciTools:masterfrom
cpelley:latest_scipy
Jan 22, 2014
Merged

BLD: Updading travis scipy package.#960
bblay merged 2 commits intoSciTools:masterfrom
cpelley:latest_scipy

Conversation

@cpelley
Copy link

@cpelley cpelley commented Jan 17, 2014

No description provided.

@cpelley
Copy link
Author

cpelley commented Jan 20, 2014

ON HOLD, currently CF site not accessible

@ghost ghost assigned bblay Jan 21, 2014
@bblay
Copy link
Contributor

bblay commented Jan 21, 2014

Ok, @cpelley, should be able to rebase onto master now travis is no longer affected by that website outage.

.travis.yml Outdated
Copy link
Member

Choose a reason for hiding this comment

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

These lines could do with a comment explaining why they are in this position in the file. (And hence when they might be able to be folded back into the normal section.)

@bblay
Copy link
Contributor

bblay commented Jan 22, 2014

Hopefully just a small change required to version number handling, fingers crossed.

@bblay
Copy link
Contributor

bblay commented Jan 22, 2014

Please could the benefits of switching to SciPy 0.12 be outlined here?
Was Travis getting 0.9 before this change?

@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

Benefits not understood by me, I don't think there is a specific user case driving it.

@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

Was Travis getting 0.9 before this change?

yes v0.9 I believe since travis is a 12.04 release of ubuntu (precise)

@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

Assuming that the above indicates a problem with the numpy version??
Making a new commit which prints out the matplotlib and numpy versions.

@bblay
Copy link
Contributor

bblay commented Jan 22, 2014

@rhattersley, what are your thoughts on building a version of SciPy for Travis and hosting it somewhere?

@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

Great, now uses the same numpy but with scipy 0.12.0

@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

Works

@bblay
Copy link
Contributor

bblay commented Jan 22, 2014

Great, now uses the same numpy but with scipy 0.12.0

Is that a problem?

@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

Great, now uses the same numpy but with scipy 0.12.0

Is that a problem?

No the opposite, using numpy from 12.04 ubuntu and scipy 0.12.0 built to use dependencies in 12.04

Thanks @esc24 for pointing us to pylab
https://launchpad.net/~pylab/+archive/stable

@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

Just rebasing now

@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

Passes, ready to go in.

.travis.yml Outdated
Copy link
Member

Choose a reason for hiding this comment

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

What's wrong with sudo add-apt-repository -y ppa:pylab/stable?

Copy link
Author

Choose a reason for hiding this comment

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

I'll give it a go

@bblay
Copy link
Contributor

bblay commented Jan 22, 2014

Please hold off merging this while to possibility of upgrading to NumPy 1.7 is explored.

@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

Please hold off merging this while to possibility of upgrading to NumPy 1.7 is explored.

Since this would be from a separate PPA, I think that should go in a separate PR

@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

Found one here:
https://launchpad.net/~tukss/+archive/ppa/+build/4055943

Its unofficial but hey, its travis and it gets wiped so we can give it a go if you like?

@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

(the one we have added for scipy is also unofficial so....)

@bblay bblay mentioned this pull request Jan 22, 2014
@bblay
Copy link
Contributor

bblay commented Jan 22, 2014

@cpelley I'm trying pip, can't do any harm for you to try launchpad, perhaps in another branch?

@bblay
Copy link
Contributor

bblay commented Jan 22, 2014

pip install -I numpy==1.7 took Travis about three minutes and the tests pass...

@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

Since travis already takes 23min... is 3 minutes significant?

@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

I'll try with the one off launchpad

bblay added a commit that referenced this pull request Jan 22, 2014
BLD: Updading travis scipy package.
@bblay bblay merged commit 132629f into SciTools:master Jan 22, 2014
@cpelley cpelley deleted the latest_scipy branch January 22, 2014 14:06
@cpelley
Copy link
Author

cpelley commented Jan 22, 2014

Update travis numpy -> v1.7.0b2
#975

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.

4 participants