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

Dropping legacy python support #1186

Closed
pelson opened this issue Nov 14, 2018 · 9 comments · Fixed by #1540
Closed

Dropping legacy python support #1186

pelson opened this issue Nov 14, 2018 · 9 comments · Fixed by #1540

Comments

@pelson
Copy link
Member

pelson commented Nov 14, 2018

We are suffering from lots of small cuts by supporting Python 2 in cartopy. As far as I can tell, none of the developers reach for py2 over py3 and this means that we are always doubling-back to ensure compatibility.

Some of the things that we are really missing out on:

The time has come to put to writing the date of a final release with python 2 support. Frankly, it isn't a sustainable model to claim that an older release will be the final "supported" python 2 release - once we make the decision, there should be no further maintenance of a py2 branch. Users can still install old versions, but as will other old releases, that would be under no expectation of maintenance / bug-fixes / support.

I'm not yet convinced that v0.17 is that release (happening this week), but I could very easily see it as the penultimate release with py2 support (thus making v0.18 / v1.0 the magic number). If that is what we want, it would be great to get that message out in the v0.17 release notes.

At the moment, this isn't a tangible proposal - I just want to start the ball rolling.

@dopplershift
Copy link
Contributor

👍 to making 0.18 the last Python 2.7 release. I’d love to get that out the door (depending on the amount of fixes/features done) in the Spring/Summer 2019 timeframe.

@QuLogic
Copy link
Member

QuLogic commented Nov 15, 2018

I'm definitely happy to see this happen at some point, especially given Matplotlib already did so. I would guess we'd like to get the fancy gridline labelling in before making the split at least.

@hugovk
Copy link
Contributor

hugovk commented Nov 15, 2018

Here's the pip installs for Cartopy from PyPI for last month, showing low numbers for Python 2:

category percent downloads
3 78.67% 4,832
2 16.53% 1,015
null 4.80% 295
Total 6,142

Source: pypistats python_major --last-month cartopy

category percent downloads
3.6 48.89% 3,003
3.5 19.70% 1,210
2.7 16.53% 1,015
3.7 8.14% 500
null 4.80% 295
3.4 1.82% 112
3.3 0.11% 7
Total 6,142

Source: pypistats python_minor --last-month cartopy

@QuLogic
Copy link
Member

QuLogic commented Nov 16, 2018

That seemed high to me initially, but turns out it's better than Matplotlib, which is already Py3-only (though there's still a bugfix branch with Py2 support):

pypistats python_major --last-month matplotlib:

category percent downloads
3 55.34% 1,835,154
2 43.93% 1,456,856
null 0.73% 24,256
Total 3,316,266

pypistats python_minor --last-month matplotlib:

category percent downloads
2.7 43.91% 1,456,264
3.6 35.55% 1,179,014
3.5 10.19% 338,008
3.7 6.82% 226,297
3.4 2.76% 91,504
null 0.73% 24,256
2.6 0.02% 592
3.3 0.00% 158
3.8 0.00% 131
3.2 0.00% 42
Total 3,316,266

@hugovk
Copy link
Contributor

hugovk commented Nov 17, 2018

Here's Matplotlib over time:

image

https://github.com/hugovk/pypi-tools

@dopplershift
Copy link
Contributor

Matplotlib going 3.x only biases download numbers, since all downloads due to upgrades (not new setups) will be Python 3 only now.

@hugovk
Copy link
Contributor

hugovk commented Dec 5, 2018

Please see PR #1237.

@pelson
Copy link
Member Author

pelson commented Jan 10, 2019

Another reason:

@hugovk
Copy link
Contributor

hugovk commented Apr 28, 2020

Please see PR #1540.

@QuLogic QuLogic added this to the 0.19 milestone Apr 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants