Skip to content

Fixes irisxr 202608 - #253

Open
pp-mo wants to merge 10 commits into
SciTools:mainfrom
pp-mo:fixes_irisxr_202608
Open

pp-mo wants to merge 10 commits into
SciTools:mainfrom
pp-mo:fixes_irisxr_202608

Conversation

@pp-mo

@pp-mo pp-mo commented Aug 15, 2026

Copy link
Copy Markdown
Member

Closes #255
Closes #258

@pp-mo pp-mo mentioned this pull request Aug 19, 2026
5 tasks
@pp-mo
pp-mo force-pushed the fixes_irisxr_202608 branch from ba5f4a8 to 7cf469f Compare September 17, 2026 17:05
@pp-mo

pp-mo commented Sep 17, 2026

Copy link
Copy Markdown
Member Author

Progress update 2026-09-19

Extensively rebased.
Should now work with forthcoming Iris 3.16.1, if it includes SciTools/iris#7281

Stiil incomplete, as more needed to get it working with latest xarray (>v2026.02.0).
Possibly, all that will be needed is to exclude all "unstructured" testfiles from integration.test_iris_xarray_roundtrips.test_roundtrip_ixi

TODO:

remove temporary test-against-private-branch code, in .github/workflows/ci-tests.yml

@pp-mo
pp-mo force-pushed the fixes_irisxr_202608 branch 2 times, most recently from d6cde1c to fec7074 Compare September 17, 2026 17:20
@pp-mo
pp-mo force-pushed the fixes_irisxr_202608 branch from fec7074 to fd44a1c Compare September 18, 2026 09:21
@pp-mo
pp-mo force-pushed the fixes_irisxr_202608 branch from fd44a1c to 9aa2ff0 Compare September 18, 2026 09:31
@pp-mo
pp-mo force-pushed the fixes_irisxr_202608 branch 2 times, most recently from 25ae3ef to 0e0389e Compare September 18, 2026 12:38
@pp-mo

pp-mo commented Sep 18, 2026

Copy link
Copy Markdown
Member Author

Removed test-against-new-iris .

@pp-mo
pp-mo force-pushed the fixes_irisxr_202608 branch from e28189f to 288b7c5 Compare September 18, 2026 12:49
@pp-mo

pp-mo commented Sep 18, 2026

Copy link
Copy Markdown
Member Author

Note:

when testing against older Iris, just 3 string-array tests fail,
all in tests/integration/test_iris_xarray_roundtrips.py :

  • test_roundtrip_ixi[ds__dtype__string]
  • test_roundtrip_ixi[ds__stringvar__singlepoint]
  • test_roundtrip_ixi[ds__stringvar__multipoint]

This is ~ as expected,

  • the dtype__string one was disabled before
  • the other 2 have been failing since xarray>v2026.02, e.g. here
  • however the Python 3.10 failures are different, as this has earlier versions :
    • Python 3.10 --> iris=3.11.1, xarray=2025.6.1 : xarray gives different string data handling
    • cf. Python 3.11 --> iris=3.14.1, xarray=2026.07
    • cf. Python 3.12/3.13 --> iris=3.15.0, xarray=2026.07

@pp-mo
pp-mo force-pushed the fixes_irisxr_202608 branch from 288b7c5 to 7135b7a Compare September 18, 2026 16:02
@pp-mo

pp-mo commented Sep 18, 2026

Copy link
Copy Markdown
Member Author

More changes

  • go back to testing against current Iris (should pass tests when 3.16 is released)
  • extend support and testing to Python 3.14

@pp-mo

pp-mo commented Sep 18, 2026

Copy link
Copy Markdown
Member Author

Expectations:

  • Pythons 3.11-3.14 should pass all tests when Iris 3.16.1 appears
  • Python 3.10 will still be failing a few things (as at present), due to (same) old versions of iris + xarray
    • our position: we do still support Python 3.10 (at least until October 2026!) , but some tests fail (some results are different)

@pp-mo
pp-mo marked this pull request as ready for review September 18, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Extend support to Python 3.14 Incompatible with iris 3.16

1 participant