Merge pull request #99 from chrishavlin/drop_py39

drop py39 support, explicitly add 3.12

Author: chrishavlin

.github/workflows/check-build.yml

@@ -6,9 +6,9 @@ jobs:
     name: twinecheck
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
     - name: Set up Python
-      uses: actions/setup-python@v4
+      uses: actions/setup-python@v5
       with:
         python-version: '3.10'
     - name: Install dependencies

.github/workflows/check-manifest.yml

@@ -7,9 +7,9 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - name: Checkout Source
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
     - name: Setup Python
-      uses: actions/setup-python@v4
+      uses: actions/setup-python@v5
       with:
         python-version: '3.10'
     - name: install check-manifest

.github/workflows/create-release.yml

@@ -13,7 +13,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
       - name: Create Release
         id: create_release
         uses: actions/create-release@v1

.github/workflows/publish-to-pypi.yml

@@ -12,9 +12,9 @@ jobs:
     runs-on: ubuntu-latest
     if: startsWith(github.ref, 'refs/tags')
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
     - name: Set up Python 3.10
-      uses: actions/setup-python@v4
+      uses: actions/setup-python@v5
       with:
         python-version: '3.10'
     - name: Install dependencies

.github/workflows/run-pytest-tests.yml

@@ -9,20 +9,22 @@ jobs:
     strategy:
       matrix:
         platform: [ubuntu-latest, windows-latest, macos-latest]
-        python-version: ['3.9', '3.10', '3.11']
+        python-version: ['3.10', '3.11', '3.12']
 
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
     - name: Set up Python ${{ matrix.python-version }}
-      uses: actions/setup-python@v4
+      uses: actions/setup-python@v5
       with:
         python-version: ${{ matrix.python-version }}
     - name: Setup yt_xarray
       run: |
         python -m pip install --upgrade pip
         python -m pip install -e .[full,test]
+    - name: List installed deps
+      run: pip list
     - name: Run Tests
-      run: pytest --cov=./ --cov-report=html
+      run: pytest --cov=./ --cov-report=html --color=yes yt_xarray
 
     - name: Upload coverage report
       if: (matrix.platform == 'ubuntu-latest' && matrix.python-version == '3.11')

.github/workflows/weekly-test.yml

@@ -15,9 +15,9 @@ jobs:
         python-version: ['3.10']
 
     steps:
-        - uses: actions/checkout@v3
+        - uses: actions/checkout@v4
         - name: Set up Python ${{ matrix.python-version }}
-          uses: actions/setup-python@v4
+          uses: actions/setup-python@v5
           with:
             python-version: ${{ matrix.python-version }}
         - name: Setup yt_xarray

pyproject.toml

@@ -9,16 +9,16 @@ authors = [
 ]
 description="interface between yt and xarray"
 readme = "README.md"
-requires-python = ">=3.9"
+requires-python = ">=3.10"
 classifiers = [
     "Development Status :: 4 - Beta",
     "Intended Audience :: Developers",
     "License :: OSI Approved :: MIT License",
     "Natural Language :: English",
     "Programming Language :: Python :: 3",
-    "Programming Language :: Python :: 3.9",
     "Programming Language :: Python :: 3.10",
     "Programming Language :: Python :: 3.11",
+    "Programming Language :: Python :: 3.12",
 ]
 dependencies=['yt>=4.2.0', 'h5py>=3.4.0', 'pooch>=1.5.1', 'xarray']
 dynamic = ["version"]

yt_xarray/tests/test_xr_to_yt.py

@@ -537,9 +537,23 @@ def test_reversed_axis(stretched, use_callable, chunksizes):
     assert np.all(np.isfinite(vals))
 
 
+def _invalid_nc4_windows():
+    import sys
+
+    import netCDF4
+
+    nc4_v = netCDF4.__version__
+    return "1.7.1" in nc4_v and sys.platform.startswith("win")
+
+
 def test_cf_xarray_disambiguation():
-    from cf_xarray.datasets import airds
 
+    if _invalid_nc4_windows():
+        # https://github.com/Unidata/netcdf4-python/issues/1373
+        # https://github.com/data-exp-lab/yt_xarray/pull/99
+        pytest.skip("Known nc4, windows endian issue.")
+
+    airds = xr.tutorial.open_dataset("air_temperature")
     # run the whole selection (will internally run coord disambiguation)
     sel = xr2yt.Selection(
         airds, fields=["air"], sel_dict={"time": 0}, sel_dict_type="isel"
@@ -554,7 +568,12 @@ def test_cf_xarray_disambiguation():
 
 
 def test_missing_cfxarray(monkeypatch):
-    from cf_xarray.datasets import airds
+    if _invalid_nc4_windows():
+        # https://github.com/Unidata/netcdf4-python/issues/1373
+        # https://github.com/data-exp-lab/yt_xarray/pull/99
+        pytest.skip("Known nc4, windows endian issue.")
+
+    airds = xr.tutorial.open_dataset("air_temperature")
 
     def _bad_import(name, globals=None, locals=None, fromlist=(), level=0):
         raise ImportError