netCDF4 is no longer a required dependency

seisman · seisman · commit e6b98f3fed3f · 2025-04-26T09:37:52.000+08:00
diff --git a/.github/workflows/benchmarks.yml b/.github/workflows/benchmarks.yml
@@ -60,7 +60,6 @@ jobs:
             numpy
             pandas
             xarray
-            netCDF4
             packaging
             geopandas
             pyarrow-core
diff --git a/.github/workflows/cache_data.yaml b/.github/workflows/cache_data.yaml
@@ -55,7 +55,6 @@ jobs:
             numpy
             pandas
             xarray
-            netCDF4
             packaging
             python-build
 
diff --git a/.github/workflows/ci_docs.yml b/.github/workflows/ci_docs.yml
@@ -96,7 +96,6 @@ jobs:
             numpy
             pandas
             xarray
-            netCDF4
             packaging
             contextily
             geopandas
diff --git a/.github/workflows/ci_doctests.yaml b/.github/workflows/ci_doctests.yaml
@@ -54,7 +54,6 @@ jobs:
             numpy
             pandas
             xarray
-            netCDF4
             packaging
             contextily
             geopandas
diff --git a/.github/workflows/ci_tests.yaml b/.github/workflows/ci_tests.yaml
@@ -75,13 +75,13 @@ jobs:
             numpy-version: '1.26'
             pandas-version: '=2.1'
             xarray-version: '=2023.07'
-            optional-packages: ' contextily geopandas ipython pyarrow-core rioxarray sphinx-gallery'
+            optional-packages: ' contextily geopandas ipython pyarrow-core rioxarray netCDF4 sphinx-gallery'
           # Python 3.13 + core packages (latest versions) + optional packages
           - python-version: '3.13'
             numpy-version: '2.2'
             pandas-version: ''
             xarray-version: ''
-            optional-packages: ' contextily geopandas ipython pyarrow-core rioxarray sphinx-gallery'
+            optional-packages: ' contextily geopandas ipython pyarrow-core rioxarray netCDF4 sphinx-gallery'
           # Python 3.12 + core packages (Linux only)
           - os: 'ubuntu-latest'
             python-version: '3.12'
@@ -129,7 +129,6 @@ jobs:
             numpy=${{ matrix.numpy-version }}
             pandas${{ matrix.pandas-version }}
             xarray${{ matrix.xarray-version }}
-            netCDF4
             packaging
             make
             pip
diff --git a/.github/workflows/ci_tests_dev.yaml b/.github/workflows/ci_tests_dev.yaml
@@ -150,8 +150,8 @@ jobs:
         run: |
           python -m pip install --pre --prefer-binary \
                         --extra-index https://pypi.anaconda.org/scientific-python-nightly-wheels/simple \
-                        numpy pandas xarray netCDF4 packaging \
-                        build contextily dvc geopandas ipython pyarrow rioxarray \
+                        numpy pandas xarray packaging \
+                        build contextily dvc geopandas ipython pyarrow rioxarray netCDF4 \
                         pytest pytest-cov pytest-doctestplus pytest-mpl pytest-rerunfailures \
                         sphinx-gallery
 
diff --git a/.github/workflows/ci_tests_legacy.yaml b/.github/workflows/ci_tests_legacy.yaml
@@ -64,7 +64,6 @@ jobs:
             numpy=1.26
             pandas
             xarray
-            netCDF4
             packaging
             contextily=1.5
             geopandas=1.0
diff --git a/.github/workflows/type_checks.yml b/.github/workflows/type_checks.yml
@@ -53,7 +53,7 @@ jobs:
           # 3. type checker and stub packages
           # 4. other packages that are used somewhere in PyGMT
           python -m pip install \
-            numpy pandas xarray netcdf4 packaging \
+            numpy pandas xarray packaging \
             contextily geopandas ipython pyarrow rioxarray \
             mypy pandas-stubs pyarrow-stubs \
             matplotlib pytest
diff --git a/ci/requirements/docs.yml b/ci/requirements/docs.yml
@@ -10,7 +10,6 @@ dependencies:
     - numpy
     - pandas
     - xarray
-    - netCDF4
     - packaging
     # Optional dependencies
     - contextily
diff --git a/doc/install.md b/doc/install.md
@@ -95,7 +95,6 @@ PyGMT requires the following packages to be installed:
 - [NumPy](https://numpy.org)
 - [pandas](https://pandas.pydata.org)
 - [Xarray](https://xarray.dev/)
-- [netCDF4](https://unidata.github.io/netcdf4-python)
 - [packaging](https://packaging.pypa.io)
 
 :::{note}
@@ -128,14 +127,14 @@ installed (we'll call it `pygmt` but feel free to change it to whatever you want
 ::: {tab-item} mamba
 :sync: mamba
 ```
-mamba create --name pygmt python=3.13 numpy pandas xarray netcdf4 packaging gmt
+mamba create --name pygmt python=3.12 numpy pandas xarray packaging gmt
 ```
 :::
 
 ::: {tab-item} conda
 :sync: conda
 ```
-conda create --name pygmt python=3.13 numpy pandas xarray netcdf4 packaging gmt
+conda create --name pygmt python=3.12 numpy pandas xarray packaging gmt
 ```
 :::
 ::::
diff --git a/environment.yml b/environment.yml
@@ -10,7 +10,6 @@ dependencies:
     - numpy>=1.26
     - pandas>=2.1
     - xarray>=2023.07
-    - netCDF4
     - packaging
     # Optional dependencies
     - contextily>=1.5
@@ -34,6 +33,8 @@ dependencies:
     - pytest-cov
     - pytest-doctestplus
     - pytest-mpl
+    # Dev dependencies (testing)
+    - netCDF4
     # Dev dependencies (building documentation)
     - geodatasets
     - myst-nb
diff --git a/pygmt/tests/test_accessor.py b/pygmt/tests/test_accessor.py
@@ -2,6 +2,7 @@
 Test the behaviour of the GMTDataArrayAccessor class.
 """
 
+import importlib
 import sys
 from pathlib import Path
 
@@ -14,14 +15,16 @@
 from pygmt.enums import GridRegistration, GridType
 from pygmt.exceptions import GMTInvalidInput
 
+_HAS_NETCDF4 = bool(importlib.util.find_spec("netCDF4"))
+
 
 def test_accessor_gridline_cartesian():
     """
     Check that the accessor returns the correct registration and gtype values for a
     Cartesian, gridline-registered grid.
     """
     fname = which(fname="@test.dat.nc", download="a")
-    grid = xr.open_dataarray(fname, engine="netcdf4")
+    grid = xr.load_dataarray(fname, engine="gmt", raster_kind="grid")
     assert grid.gmt.registration == GridRegistration.GRIDLINE
     assert grid.gmt.gtype == GridType.CARTESIAN
 
@@ -32,7 +35,7 @@ def test_accessor_pixel_geographic():
     geographic, pixel-registered grid.
     """
     fname = which(fname="@earth_relief_01d_p", download="a")
-    grid = xr.open_dataarray(fname, engine="netcdf4")
+    grid = xr.load_dataarray(fname, engine="gmt", raster_kind="grid")
     assert grid.gmt.registration == GridRegistration.PIXEL
     assert grid.gmt.gtype == GridType.GEOGRAPHIC
 
@@ -104,6 +107,7 @@ def test_accessor_set_invalid_registration_and_gtype():
 
 
 # TODO(GMT>=6.5.0): Remove the xfail marker for GMT>=6.5.0.
+@pytest.mark.skipif(condition=not _HAS_NETCDF4, reason="netCDF4 is not installed")
 @pytest.mark.xfail(
     condition=sys.platform == "win32" and Version(__gmt_version__) < Version("6.5.0"),
     reason="Upstream bug fixed in https://github.com/GenericMappingTools/gmt/pull/7573",
@@ -121,7 +125,7 @@ def test_accessor_sliced_datacube():
             "https://github.com/pydata/xarray-data/raw/master/eraint_uvz.nc",
             download="u",
         )
-        with xr.open_dataset(fname) as dataset:
+        with xr.open_dataset(fname, engine="netcdf4") as dataset:
             grid = dataset.sel(level=500, month=1, drop=True).z
 
         assert grid.gmt.registration == GridRegistration.GRIDLINE
diff --git a/pygmt/tests/test_io.py b/pygmt/tests/test_io.py
@@ -9,6 +9,8 @@
 from pygmt.helpers import GMTTempFile
 from pygmt.io import load_dataarray
 
+pytest.importorskip("netCDF4")
+
 
 @pytest.mark.benchmark
 def test_io_load_dataarray():
diff --git a/pyproject.toml b/pyproject.toml
@@ -38,7 +38,6 @@ dependencies = [
     "numpy>=1.26",
     "pandas>=2.1",
     "xarray>=2023.07",
-    "netCDF4",
     "packaging",
 ]
 dynamic = ["version"]
diff --git a/requirements.txt b/requirements.txt
@@ -2,5 +2,4 @@
 numpy>=1.26
 pandas>=2.1
 xarray>=2023.07
-netCDF4
 packaging

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,6 @@ dependencies = [`
`38`	`38`	`"numpy>=1.26",`
`39`	`39`	`"pandas>=2.1",`
`40`	`40`	`"xarray>=2023.07",`
`41`		`- "netCDF4",`
`42`	`41`	`"packaging",`
`43`	`42`	`]`
`44`	`43`	`dynamic = ["version"]`