Skip to content

BLD: fix 2.7_LOCALE build #15540

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants