LLNL / UEDGE

2D fluid simulation of plasma and neutrals in magnetic fusion devices

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Moving to python 3

sballin opened this issue · comments

Here are some reasons I think we would benefit from dropping support for python 2:

  • Python 2 relies on dead code: the important libraries on this list (including numpy and matplotlib) have pledged to stop updating their python 2 versions before 2020
  • Uniform codebase: right now we have a mix of routines, some designed for python 2, others for python 3. Sometimes when you want to use a new script, you need to port it yourself.
  • Simplified codebase: remove code that does different things depending on the python version. A smaller and simpler codebase is a more maintainable codebase.
  • Old python 2 code would not be gone forever: it's still in the git history if anyone needs to use it

Drawbacks:

  • In my experience, Totalview can only show the Fortran source when uedge is used with python 2

I think this task would be pretty simple using the 2to3 utility, and I would be happy to make a pull request.

Let me know your thoughts on this, and if there are any drawbacks I'm unaware of.

Closed as resolved: UEDGE no longer supports Python 2