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

BLD: fix 2.7_LOCALE build #15540

Closed
wants to merge 1 commit into from
Closed

BLD: fix 2.7_LOCALE build #15540

wants to merge 1 commit into from

Conversation

jreback
Copy link
Contributor

@jreback jreback commented Mar 1, 2017

simplify install_travis.sh, always using a clean miniconda install
bump matplotlib to 1.4.0 for _LOCALE, numpy to 1.8.2

So this removes our testing of mpl 1.2.1 with numpy 1.7.1. We still test
elsewhere 1.7.1, and mpl 1.3.1 (with 1.8.2), so I think this is sufficient.

something changed in conda meta data to make the installtion of numpy=1.7.1 / mpl=1.2.1 fail
https://travis-ci.org/jreback/pandas/jobs/206680630

The only question is to we want to 'officially' say that we require mpl >= 1.3.1 ?
Things will still work as long as the installation succeeds, but travis is being problematic.

cc @jorisvandenbossche
cc @tacaswell

@jreback jreback added the CI Continuous Integration label Mar 1, 2017
@jreback jreback added this to the 0.20.0 milestone Mar 1, 2017
simplify install_travis.sh, always using a clean miniconda install

bump matplotlib to 1.4.0 for _LOCALE, numpy to 1.8.2
@jreback
Copy link
Contributor Author

jreback commented Mar 1, 2017

merging to avoid constant travis failures, but please comment on the question I raised above.

@jreback jreback closed this in 2340fb8 Mar 1, 2017
@jorisvandenbossche
Copy link
Member

I have no problem with bumping the minimal matplotlib version from 1.2 to > 1.3.1 (1.3 dates from 2013 I think, so we might even go to 1.4? cc @TomAugspurger @sinhrks ).

Should probably open a separate issue for that to remind of code clean-up and update of docs.

@jorisvandenbossche
Copy link
Member

Ah, I see that in practice > 1.3.1 actually means >= 1.4 (1.4.0 dates from August 2014). But as I said, that sounds good to me to bump to that.

AnkurDedania pushed a commit to AnkurDedania/pandas that referenced this pull request Mar 21, 2017
simplify install_travis.sh, always using a clean miniconda install
bump matplotlib to 1.4.0 for _LOCALE, numpy to 1.8.2

So this removes our testing of mpl 1.2.1 with numpy 1.7.1.
We *still* test elsewhere 1.7.1, and mpl 1.3.1 (with 1.8.2)

Author: Jeff Reback <jeff@reback.net>

Closes pandas-dev#15540 from jreback/build and squashes the following commits:

58b6f2f [Jeff Reback] BLD: fix 2.7_LOCALE build
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants