-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Matplotlib manylinux wheel - ready to ship? #6473
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
Comments
Given that a 1.5.2 release is imminent and the changes to support many wheels got backported, I would rather only ship the wheels for 1.5.2 onward. Thank you for your tireless work on this. |
My reasoning for not replacing old releases is the complaints numpy got for doing the same thing. |
We did get one complaint (numpy/numpy#7570) but that user was doing something rather weird in his setup (strip of the binaries). Somebody doing the same thing will hit the same problem when 1.5.2 comes out (the complaint was from someone pinning to an old numpy version in the hope that would always be source-only). On the other hand, having binaries for old releases on pypi makes it much easier to test against earlier versions. Maybe worth a straw poll on the list? |
👍 to a straw poll. I would start with the matplotlib-devel. |
Well, I don't think we got any interesting information from straw poll, it's quite possible that no-one has tested the wheels. Any more thoughts here? I still have a preference for uploading the wheels, and I'm happy for you to make the call either way. |
It seems 1.5.1 was shipped without manylinux wheels: https://pypi.python.org/pypi/matplotlib/1.5.1 |
1.5.2 should ship with manylinux wheels. The release of 1.5.1 predates the manylinux wheels |
I'm still advocating shipping a manylinux wheel for 1.5.1 - but I think someone has to pronounce on whether that should happen. Built wheels for testing at https://nipy.bic.berkeley.edu/manylinux . |
Wouldn't it require adding #6442 to 1.5.1 then IMHO it can't be released as 1.5.1 but would have to be 1.5.1.1 or some such thing? |
Right - the wheels I built and uploaded at https://nipy.bic.berkeley.edu/manylinux are patched with a version of #6442. I thought that, given this should give no behavior change, and only affected how matplotlib gets its symbols from the relevant libraries, this might be acceptable as part of the wheel build process, but maybe that's a stretch. |
The above is just my opinion others may feel differently. Ant to be clear I am very happy to see the wheels get published as soon as possible. |
When is 1.5.2 planned to be released? |
I intend to do the 1.5.2 release July 2. |
That's 3-4 days away... My 2 cents: Users can stay without wheels for 4 days ^_^ |
Sure. I was proposing the older wheels mainly for CI against older mpl versions. It makes it easier if these are on pypi (no extra URL to use, fast reliable downloads). But, it's only a gentle suggestion. |
I think CI is probably less important for 1.5.1 vs. 1.5.2 than for say 1.4 vs. 1.5 |
Yes, I am more interested in having 1.4 and 1.3 to test against. |
not to totally hijack this thread - but is there anything blocking manylinux wheels for 1.4? i.e., if I try to compile them will it work? |
Yes, 1.4 wheels will work - see the recipe in https://github.com/matthew-brett/manylinux-builds/blob/master/build_matplotlibs.sh . |
If we are going to do 1.4 and 1.3 wheels (which people should stop using, we do not fix bugs in those series anymore!) we should probably backport the tk fixes and do a new micro for each of them? It is very non-standard, but so is pip growing the ability to distribute linux wheels. |
I don't understand--why distribute wheels for old versions? Is it really worth the effort? |
My understanding of the use case is like we test against a very old version of numpy, down-stream to us projects what to test against old versions of mpl and the wheels would save them a whole bunch of CI time. |
I think it's totally fair for you folks NOT to distribute 1.4 and 1.3 wheels and instead encourage people to move to more modern matplotlib versions. I just wanted to confirm that if I made them myself it would be okay. |
A bit off topic, but just wondering if/when 1.5.2 will be shipped in pip? I see the release happened on github back on July 3rd. Cheers. |
Sorry to keep on - but I'd really like to put a wheel up for a course I'm
doing in 10 days. Any chance that would be possible?
|
I will do the source release tomorrow. On Wed, Aug 17, 2016, 20:24 Matthew Brett notifications@github.com wrote:
|
Thank you - that is very helpful. |
Will this be the source from the current tag v1.5.2 (commit 179de2b)? |
Yes On Wed, Aug 17, 2016 at 9:11 PM Matthew Brett notifications@github.com
|
OK to upload wheels today? |
Is it going to be tagged 1.5.2 or 1.5.3? On Wed, Aug 17, 2016 at 9:53 PM, Matthew Brett notifications@github.com
|
I am going to push up a 1.5.2 today as there seems to be demand for it and get to 1.5.3 when I get to it. |
@matthew-brett The source is up, please close this when the wheels are up. Sorry doing being so slow on my end. |
@tacaswell - no problem - wheels uploaded now.
For the next release, here's a humble plea to put up the wheels first, so that no-one accidentally hits a source install when they were expecting a wheel install, in the gap between the source and the wheel upload. |
As y'all may have seen, I've been working on building manylinux wheels for matplotlib
I have an automated build script for matplotlib 1.5.1, that adds a version of PR #6442 to enable tkagg.
Thanks to Thomas' help over at #6469, these wheels are passing all tests on a fresh install into travis-ci containers.
Olivier Grisel has installed and tested on his Ubuntu 14.04 machine. I have also installed and tested on a Debian Jessie VM.
I think these wheels are now ready to deploy on pypi. Do y'all agree? Is there anything else I should build into these wheels (see the build script link above)? Are there any other tests I / we can do?
The text was updated successfully, but these errors were encountered: