DendroPy icon indicating copy to clipboard operation
DendroPy copied to clipboard

Keep supporting Python2.7?

Open mmore500 opened this issue 2 years ago • 2 comments

Looks like there is one line of code that uses fstring syntax, which is a Python3.6+ only feature.

https://github.com/jeetsukumaran/DendroPy/blob/ad33b71b8c8044139434c584add580864e97b6d5/src/dendropy/model/coalescent.py#L731

Not sure what the fraction of Python2 looks like among the user base. Because it's just one line, I'll push a patch for Python2-friendly syntax on a branch I'm working on. For now, infrastructure to install Python2.7 for testing isn't too hard to manage, so that's not a strong factor to suggest dropping Python2.7 support either. Figured I'd open this issue to check in on the thinking/plan on this front for the future.

mmore500 avatar Mar 31 '23 22:03 mmore500

That f-string usage was probably a bit careless of me. At the same time, I'm personally happy dropping support for 2.7 going forward. I think maintaining a 2.x compatible branch may lead to lots of headaches, my personal preference is to just go 3.x and not look back?

jeetsukumaran avatar Apr 01 '23 21:04 jeetsukumaran

I think that would make a lot of sense. Especially because Python2 has been EOL'ed for 3 years now and users will still be able to access previous versions (just not track new features).

Because the current codebase does have python2 support it might make sense to do one more release and then drop python support promises from the documentation and package manifest?

mmore500 avatar Apr 01 '23 21:04 mmore500

Python2.7 support will be dropped in Dendropy 5.

mmore500 avatar May 18 '24 08:05 mmore500