SciTools / iris-esmf-regrid

A collection of structured and unstructured ESMF regridding schemes for Iris.
https://iris-esmf-regrid.readthedocs.io/en/latest
BSD 3-Clause "New" or "Revised" License
20 stars 17 forks source link

Enable Python 3.9 and 3.10 #161

Closed zklaus closed 2 years ago

zklaus commented 2 years ago

✨ Feature Request

It would be great if we could get Python 3.9 and Python 3.10 builds for iris-esmf-regrid. At the moment, the maximum version is 3.8, but it is unclear to me whether that is simply because that was the version available when the CI was setup or for a deeper reason.

Does anyone know?

trexfeathers commented 2 years ago

@zklaus we're currently held up by SciTools/iris#4465, which is a barrier to advancing Iris (and therefore iris-esmf-regrid too) beyond 3.8.

zklaus commented 2 years ago

Thanks, @trexfeathers for the explanation. However, Iris itself doesn't seem to draw the same conclusion and installs happily in 3.9 and 3.10.

trexfeathers commented 2 years ago

@zklaus the problem is that we're unable to run Iris' test suite >3.8, so we therefore can't officially support >3.8

zklaus commented 2 years ago

I understand. But nevertheless, iris installs in 3.9 and 3.10 environments, so I don't think that should be grounds for iris-esmf-regrid to forbid it.

github-actions[bot] commented 2 years ago

@SciTools-incubator/esmf-regrid-devs This issue is stale due to a lack of activity in the last 90 days. Remove stale label or comment, otherwise this issue will close automatically in 7 days time.

trexfeathers commented 2 years ago

@stephenworsley thoughts?

github-actions[bot] commented 2 years ago

@SciTools-incubator/esmf-regrid-devs This issue is stale due to a lack of activity in the last 90 days. Remove stale label or comment, otherwise this issue will close automatically in 7 days time.

stephenworsley commented 2 years ago

Since iris now has these requirements files, these can be added to iris-esmf-regrid without any problem.

zklaus commented 2 years ago

My understanding is that this was fixed with #208, right?

Thanks, @stephenworsley, @trexfeathers! :tada:

stephenworsley commented 2 years ago

Closed by #208