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

Upgrade python version to 3.8 #294

Closed
wants to merge 2 commits into from
Closed

Conversation

hassec
Copy link
Contributor

@hassec hassec commented Feb 27, 2024

During debugging of the pipeline failure in #293, I noticed that the errors actually seem to come from the upgrade to python 3.8.

This PR currently only upgrades the python version to 3.8, which is a prerequisite for #293.

Before I spent more time on this I'd like to understand if an upgrade of the python version is an option for you. 3.6 is by now quite old and was end of life in 2021, and even 3.8 is going to be end of life this October, see here

@orso82
Copy link
Member

orso82 commented Feb 27, 2024

I'd like @smithsp @kalling to comment on this. In general I think it's better to be compatible with more versions of Python, rather than fewer, unless of course there is some gain from upgrading.

@hassec what do you think is the benefit of making OMAS compatible with Python 3.8 and up, instead of 3.6 and up?

@hassec
Copy link
Contributor Author

hassec commented Feb 27, 2024

@orso82 in general I agree that there is no harm in supporting older python versions. But it is a tradeoff as it usually also means one can't use newer features like.

Requiring python >= 3.8 happened in matplotlib 2 years ago and for recent matplotlib versions it's already >= 3.9. Similar things are true for numpy which is currently >= 3.9 and went to >=3.8 in August of 2021.

I'd say going to >= 3.9 would be a nice step, and then we could even consider fixing the tests and running the CI for 3.9, 3.10, 3.11, and 3.12 to also test for all of these versions.

@kalling
Copy link

kalling commented Feb 27, 2024

Given their tight coupling, until OMFIT uses >=3.8 we probably don't want to restrict omas to >=3.8
OMFIT is currently 3.7
Updating to newer versions of Python for OMFIT is in progress. A conda package with partial support for 3.8 was put on the omfit channel years ago as a test, but there are certain dependencies (Like MDSplus) that don't currently have an appropriate newer conda package nor an easy way to build them. After we have a good process for those errant dependencies, keeping up with current versions of Python in the future should be much more straight forward.

@smithsp
Copy link
Member

smithsp commented Feb 27, 2024

Per https://anaconda.org/conda-forge/mdsplus/files we have MDSplus for various versions of python.

@kalling
Copy link

kalling commented Feb 27, 2024

@smithsp Yes, though only for Linux for recent versions of MDSplus.

@smithsp
Copy link
Member

smithsp commented Feb 28, 2024

@kalling Good catch. I need to go back and see why it was failing on osx, to see if it has been resolved with recent changes to the MDSplus build process.

@hassec hassec closed this Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants