Merged
Conversation
move 3.5 install line in travis build for pure install reconfigure travis setup to use pip for some things make pandas versions explicit in req files add numba to non min builds remove numba from 3.5 since it is not yet supported move scipy to setup.py. remove from conda ymls switch from setup.py to pip install . add path echo add ls lib add atlas try installing scipy using apt move np/pd earlier in build remove scipy as dependency. add scipy testing to travis script. remove cruft. simplify pip scipy install command remove numba from py34. update readme
This was referenced Oct 26, 2015
Closed
Member
Author
|
I going ahead with this but will revert or fix if there are any future objections. |
wholmgren
added a commit
that referenced
this pull request
Oct 27, 2015
Python 3.5, reconfigure Travis
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
1 participant
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 PR primarily does two things:
I originally thought that there was an issue with scipy/linux/python3, but it turns out that it's actually a problem with the version of gcc that Continuum uses to build scipy. For more, see ContinuumIO/anaconda-issues#425. The Readme text has been updated accordingly.
The Travis build matrix was getting to be too big and take too long, so this PR only tests the minimum pandas/numpy version on Python 2.7, and the latest versions for Python 2.7, 3.4, and 3.5. I think it is reasonable to say that pvlib is supported on the intermediate versions (Python 3.3, pandas 0.14-0.16) even if we're not explicitly testing for it.
One thing that I'm no longer testing is the numba accelerated SPA code on Python 3. It is still tested on Python 2.7. I'll change it once Continuum gets their SciPy packages fixed. I've already spent way too much time on this PR to hack this together too.
@bmu @Calama-Consulting I will merge this in ~24 hours if nobody objects. I think it's also worth the effort to make a minor release later this week.
Closes #87.