Repository navigation
Change travis to use wheel binaries - #1029
Merged
Merged
Conversation
Member
Author
|
Oh, and I should also write a page in the docs about how to build the wheels, but this change is helpful enough in speeding all the builds that I think it's better to get this in now and add the docs on making wheels later. |
Contributor
|
+1 to both docs later & speedups sooner; it would be particularly nice if the wheels are hosted on a server that non-astropy projects could use (say, those which aren't yet ready to try to become affiliated). |
2 tasks
Member
Author
|
Alright, this appears to work with the new location, so I'm going to go ahead and merge this. @mdboom - if you have thoughts on how to optimize the open file detection, we can work on that in a separate issue. |
eteq
added a commit
that referenced
this pull request
May 4, 2013
Change travis to use wheel binaries
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This was discussed at length in #938 . The basic idea is that our travis tests run slowly for a variety of reasons, but one of the important ones is that it has to build everything from source. This PR changes the travis behavior to instead install numpy, scipy, and Cython from binaries (in the new "wheel" format - see PEP427).
Once the tests on this run we'll see what the speedup is exactly, but generally I've been getting < 1 hr total for all of the tests on this, vs ~2.5 hr for the current master branch. Also, as a bonus, the build/test log for astropy is now much simpler, because it doesn't have all the crap from scipy and numpy getting built.
This is also enhanced from previous versions discussed in #938 in that it will fall back on the build-from-source methodology if the wheel fails to install for whatever reason. That will of course slow everything down, but at least they don't just all fail if there's a web server hiccup or something.
A few items remain before this can get merged, though:
(cc @iguananaut @astrofrog @keflavich as interested parties from #938)