-
Notifications
You must be signed in to change notification settings - Fork 677
TST: Add pandas wheels to Travis build #313
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
Conversation
needs a rebase also why don't we xfail the failing tests for the moment, just to get a green build (and create an issue to revert this). |
@jreback : Rebase is done, but I want to fix up the |
|
.travis.yml
Outdated
# installing the correct versions. | ||
- if [[ "$PANDAS" == "MASTER" ]]; then | ||
conda install pandas; | ||
PRE_WHEELS="https://7933911d6844c6c53a7d-47bd50c35cd79bd838daf386af554a83.ssl.cf2.rackcdn.com"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pip install numpy pytz python-dateutil
here, remove the conda install pandas
and this should work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would go with conda install numpy pytz python-dateutil
, but yes, point well taken.
@jreback : No longer an issue it seems. Also, FYI, when I install from the daily builds, it also pulls in the daily |
see my comment.when using pip, you need to satisfy its deps first. |
@jreback : Not necessarily, as |
Add pandas 0.20.1 and master to Travis. Drop 0.16.2 and Python 3.4 machines. Closes gh-307.
@jreback : I fixed the installation for the |
prob should say only supports >= 0.17.0 or something. can you add a whatsnew/0.4.0 (and add this). |
@jreback : Sounds good will do (also drop Python 3.4). |
0.20.1
andmaster
to Travis.0.16.2
and Python 3.4 machines.Closes #307.