Run pre-commit on all.

pp-mo · pp-mo · commit 88a459c08ee6 · 2025-08-14T15:04:42.000+01:00
diff --git a/lib/ncdata/__init__.py b/lib/ncdata/__init__.py
@@ -8,6 +8,7 @@
 thus acting as an efficient exchange channel between any of those forms.
 
 """
+
 # N.B. this file excluded from isort, as we want a specific class order for the docs
 
 from ._core import NameMap, NcAttribute, NcData, NcDimension, NcVariable
diff --git a/lib/ncdata/_core.py b/lib/ncdata/_core.py
@@ -11,6 +11,7 @@
 of dimensions referenced by variables.
 
 """
+
 from functools import wraps
 from typing import (
     Any,
@@ -377,7 +378,7 @@ def _addlines_indent(text, indent=""):
 
 
 # common indent spacing
-_indent = " " * 4
+_STANDARD_INDENT = " " * 4
 
 
 class NcData(_AttributeAccessMixin):
@@ -420,7 +421,7 @@ def _print_content(self) -> str:
         class, so it isn't technically an abstract method).
         This "NcData._print_content()" is called recursively for groups.
         """
-        global _indent
+        global _STANDARD_INDENT  # noqa: F824
         # Define a header line (always a separate line).
         noname = "<'no-name'>"
         lines = [f"<NcData: {self.name or noname}"]
@@ -431,20 +432,20 @@ def _print_content(self) -> str:
             if len(els):
                 if eltype == "attributes":
                     # Attributes a bit different: #1 add 'globol' to section title.
-                    lines += [f"{_indent}global attributes:"]
+                    lines += [f"{_STANDARD_INDENT}global attributes:"]
                     # NOTE: #2 show like variable attributes, but *no parent name*.
                     attrs_lines = [
                         f":{attr._print_content()}"
                         for attr in self.attributes.values()
                     ]
                     lines += _addlines_indent(
-                        "\n".join(attrs_lines), _indent * 2
+                        "\n".join(attrs_lines), _STANDARD_INDENT * 2
                     )
                 else:
-                    lines += [f"{_indent}{eltype}:"]
+                    lines += [f"{_STANDARD_INDENT}{eltype}:"]
                     for el in els.values():
                         lines += _addlines_indent(
-                            el._print_content(), _indent * 2
+                            el._print_content(), _STANDARD_INDENT * 2
                         )
                 lines.append("")
 
@@ -589,7 +590,7 @@ def __init__(
     #     return self.data.shape
 
     def _print_content(self):
-        global _indent
+        global _STANDARD_INDENT  # noqa: F824
         dimstr = ", ".join(self.dimensions)
         typestr = str(self.dtype) if self.dtype else "<no-dtype>"
         hdr = f"<NcVariable({typestr}): {self.name}({dimstr})"
@@ -602,7 +603,7 @@ def _print_content(self):
                 f"{self.name}:{attr._print_content()}"
                 for attr in self.attributes.values()
             ]
-            lines += _addlines_indent("\n".join(attrs_lines), _indent)
+            lines += _addlines_indent("\n".join(attrs_lines), _STANDARD_INDENT)
             lines += [">"]
         return "\n".join(lines)
 
diff --git a/lib/ncdata/dataset_like.py b/lib/ncdata/dataset_like.py
@@ -15,7 +15,7 @@
 :class:`~ncdata.NcVariable` objects,  but emulate the access APIs of
 :class:`netCDF4.Dataset` :class:`netCDF4.Dimension` and :class:`netCDF4.Variable`.
 
-This is provided primarily to support a re-use of the :mod:`iris.fileformats.netcdf`
+This is provided primarily to support the reuse of the :mod:`iris.fileformats.netcdf`
 file format load + save, to convert cubes to + from ncdata objects (and hence,
 especially, to convert Iris :class:`~iris.cube.Cube`\s to + from an Xarray
 :class:`~xarray.Dataset`
@@ -29,6 +29,7 @@
 complete, so this module may need to be extended, in future, to support other such uses.
 
 """
+
 from typing import Any, Dict, List
 
 import dask.array as da
diff --git a/lib/ncdata/iris.py b/lib/ncdata/iris.py
@@ -4,6 +4,7 @@
 Convert :class:`~ncdata.NcData`\s to and from Iris :class:`~iris.cube.Cube`\s.
 
 """
+
 from typing import Any, AnyStr, Dict, Iterable, List, Union
 
 #
diff --git a/lib/ncdata/iris_xarray.py b/lib/ncdata/iris_xarray.py
@@ -9,6 +9,7 @@
 the relevant load/save routines.
 
 """
+
 import xarray
 from iris.cube import CubeList
 
diff --git a/lib/ncdata/netcdf4.py b/lib/ncdata/netcdf4.py
@@ -4,6 +4,7 @@
 Converts :class:`ncdata.NcData` to and from :class:`netCDF4.Dataset` objects.
 
 """
+
 from pathlib import Path
 from threading import Lock
 from typing import Dict, Optional, Union
diff --git a/lib/ncdata/threadlock_sharing.py b/lib/ncdata/threadlock_sharing.py
@@ -49,6 +49,7 @@
     thread-safe.  So probably can only be applied at the outer level of an operation.
 
 """
+
 from contextlib import contextmanager
 from unittest import mock
 
diff --git a/lib/ncdata/utils/__init__.py b/lib/ncdata/utils/__init__.py
@@ -1,4 +1,5 @@
 """General user utility functions."""
+
 from ._compare_nc_datasets import dataset_differences, variable_differences
 from ._copy import ncdata_copy
 from ._save_errors import save_errors
diff --git a/lib/ncdata/utils/_save_errors.py b/lib/ncdata/utils/_save_errors.py
@@ -1,4 +1,5 @@
 """User utility routines for ncdata."""
+
 from typing import Dict, List, Union
 
 import netCDF4 as nc
diff --git a/tests/__init__.py b/tests/__init__.py
@@ -1,4 +1,5 @@
 """Tests for :mod:`ncdata`."""
+
 from pathlib import Path
 
 testdata_dir = Path(__file__).parent / "testdata"
diff --git a/tests/data_testcase_schemas.py b/tests/data_testcase_schemas.py
@@ -9,6 +9,7 @@
 specs.  This enables us to perform various translation tests on standard testfiles from
 the Iris and Xarray test suites.
 """
+
 from dataclasses import dataclass
 from pathlib import Path
 from typing import Dict, Union
@@ -298,7 +299,7 @@ def make_testcase_dataset(filepath, spec):
 }
 
 # Define a sequence of standard testfile specs, with suitable param-names.
-_Standard_Testcases: Dict[str, Union[Path, dict]] = {}
+_STANDARD_TESTCASES: Dict[str, Union[Path, dict]] = {}
 
 
 # A decorator for spec-generating routines.
@@ -311,8 +312,7 @@ def standard_testcases_func(func):
     A decorator for spec-generating routines.  It automatically **calls** the wrapped
     function, and adds the results into the global "_Standard_Testcases" dictionary.
     """
-    global _Standard_Testcases
-    _Standard_Testcases.update(func())
+    _STANDARD_TESTCASES.update(func())
     return func
 
 
@@ -473,7 +473,7 @@ class TestcaseSchema:
     filepath: Path = None
 
 
-@pytest.fixture(params=list(_Standard_Testcases.keys()))
+@pytest.fixture(params=list(_STANDARD_TESTCASES.keys()))
 def standard_testcase(request, session_testdir):
     """
     Provide a set of "standard" dataset testcases.
@@ -489,7 +489,7 @@ def standard_testcase(request, session_testdir):
     For those not based on a spec, 'spec' is None.
     """
     name = request.param
-    spec = _Standard_Testcases[name]
+    spec = _STANDARD_TESTCASES[name]
     if isinstance(spec, dict):
         # Build a temporary testfile from the spec, and pass that out.
         filepath = session_testdir / f"sampledata_{name}.nc"
diff --git a/tests/integration/equivalence_testing_utils.py b/tests/integration/equivalence_testing_utils.py
@@ -4,12 +4,12 @@
 Used by routines in tests/integration which attempt to show that conversions between
 ncdata and other types of data preserve information.
 """
+
 import dask.array as da
+import iris.mesh
 import numpy as np
 import pytest
 
-import iris.mesh
-
 
 def cubes_equal__corrected(c1, c2):
     """
@@ -127,7 +127,6 @@ def adjust_chunks():
     """
     import dask.config as dcfg
 
-    global _USE_TINY_CHUNKS, _CHUNKSIZE_SPEC
     if _USE_TINY_CHUNKS:
         with dcfg.set({"array.chunk-size": _CHUNKSIZE_SPEC}):
             yield
diff --git a/tests/integration/example_scripts/ex_dataset_print.py b/tests/integration/example_scripts/ex_dataset_print.py
@@ -1,4 +1,5 @@
 """Temporary integrational proof-of-concept example for dataset printout."""
+
 import iris
 
 import ncdata.iris as nci
diff --git a/tests/integration/example_scripts/ex_iris_saveto_ncdata.py b/tests/integration/example_scripts/ex_iris_saveto_ncdata.py
@@ -3,6 +3,7 @@
 
 Check that conversion succeeds and print the resulting dataset.
 """
+
 import iris
 
 from ncdata.iris import from_iris
diff --git a/tests/integration/example_scripts/ex_iris_xarray_conversion.py b/tests/integration/example_scripts/ex_iris_xarray_conversion.py
@@ -3,6 +3,7 @@
 
 Showing conversion from Xarray to Iris, and back again.
 """
+
 import dask.array as da
 import iris
 import numpy as np
diff --git a/tests/integration/example_scripts/ex_ncdata_netcdf_conversion.py b/tests/integration/example_scripts/ex_ncdata_netcdf_conversion.py
@@ -3,6 +3,7 @@
 
 Showing loading and saving ncdata to/from netcdf files.
 """
+
 import tempfile
 from pathlib import Path
 from shutil import rmtree
diff --git a/tests/integration/test_iris_load_and_save_equivalence.py b/tests/integration/test_iris_load_and_save_equivalence.py
@@ -5,6 +5,7 @@
 (1) check equivalence of cubes : iris.load(file) VS iris.load(ncdata(file))
 (2) check equivalence of files : iris -> file VS iris->ncdata->file
 """
+
 from subprocess import check_output
 
 import iris
diff --git a/tests/integration/test_iris_xarray_roundtrips.py b/tests/integration/test_iris_xarray_roundtrips.py
@@ -5,6 +5,7 @@
 (1) check equivalence of cubes : iris.load(file) VS iris.load(ncdata(file))
 (2) check equivalence of files : iris -> file VS iris->ncdata->file
 """
+
 from subprocess import check_output
 
 import dask.array as da
diff --git a/tests/integration/test_netcdf_roundtrips.py b/tests/integration/test_netcdf_roundtrips.py
@@ -1,6 +1,7 @@
 """
 Test ncdata.netcdf by checking load-save roundtrips for standard testcases.
 """
+
 from subprocess import check_output
 
 from ncdata.netcdf4 import from_nc4, to_nc4
diff --git a/tests/integration/test_xarray_load_and_save_equivalence.py b/tests/integration/test_xarray_load_and_save_equivalence.py
@@ -5,6 +5,7 @@
 (1) check equivalence of cubes : xarray.load(file) VS xarray.load(ncdata(file))
 (2) check equivalence of files : xarray -> file VS xarray->ncdata->file
 """
+
 import pytest
 import xarray
 
diff --git a/tests/test_samplecode_cdlgen_comparablecdl.py b/tests/test_samplecode_cdlgen_comparablecdl.py
@@ -5,6 +5,7 @@
 
 The status and usage of this are yet to be determined.
 """
+
 import os
 import subprocess
 from pathlib import Path
diff --git a/tests/unit/core/test_AttributeAccessMixin.py b/tests/unit/core/test_AttributeAccessMixin.py
@@ -4,6 +4,7 @@
 Note: actually tested via the inheriting classes NcData and NcVariable.
 All tests are run for both of those.
 """
+
 import numpy as np
 import pytest
 
diff --git a/tests/unit/core/test_AttrvalsDict.py b/tests/unit/core/test_AttrvalsDict.py
@@ -1,4 +1,5 @@
 """Tests for class :class:`ncdata._core.AttrvalsDict`."""
+
 from copy import deepcopy
 
 import numpy as np
diff --git a/tests/unit/core/test_NameMap.py b/tests/unit/core/test_NameMap.py
@@ -1,6 +1,7 @@
 """
 Tests for class :class:`ncdata.NameMap`.
 """
+
 from copy import deepcopy
 
 import pytest
diff --git a/tests/unit/core/test_NcAttribute.py b/tests/unit/core/test_NcAttribute.py
@@ -3,6 +3,7 @@
 
 Very simple for now, but we may add more behaviour in future.
 """
+
 import numpy as np
 import pytest
 
diff --git a/tests/unit/core/test_NcData.py b/tests/unit/core/test_NcData.py
@@ -2,6 +2,7 @@
 
 There is almost no behaviour, but we can test some constructor usages.
 """
+
 from ncdata import NcData, NcDimension, NcVariable
 
 
diff --git a/tests/unit/core/test_NcDimension.py b/tests/unit/core/test_NcDimension.py
@@ -1,4 +1,5 @@
 """Tests for class :class:`ncdata.NcDimension`."""
+
 import numpy as np
 import pytest
 
diff --git a/tests/unit/core/test_NcVariable.py b/tests/unit/core/test_NcVariable.py
@@ -3,6 +3,7 @@
 There is almost no actual behaviour, but we can test some constructor behaviours,
 such as valid calling options and possibilities for data and dtype.
 """
+
 import dask.array as da
 import numpy as np
 import pytest
diff --git a/tests/unit/iris/test_from_iris.py b/tests/unit/iris/test_from_iris.py
@@ -8,6 +8,7 @@
 This module only tests some specific API and behaviours of the top-level function, not
 covered by the generic 'roundtrip' testcases.
 """
+
 from unittest.mock import patch
 
 import dask.array as da
diff --git a/tests/unit/iris/test_to_iris.py b/tests/unit/iris/test_to_iris.py
@@ -8,6 +8,7 @@
 This module only tests some specific API and behaviours of the top-level function, not
 covered by the generic 'roundtrip' testcases.
 """
+
 import dask.array as da
 import numpy as np
 from iris._constraints import NameConstraint
diff --git a/tests/unit/netcdf/test_from_nc4.py b/tests/unit/netcdf/test_from_nc4.py
@@ -8,6 +8,7 @@
 This module only tests some specific API of the top-level access function, not covered
 by the generic 'roundtrip' testcases.
 """
+
 from pathlib import Path
 
 import netCDF4 as nc
diff --git a/tests/unit/netcdf/test_to_nc4.py b/tests/unit/netcdf/test_to_nc4.py
@@ -8,6 +8,7 @@
 This module tests specific API properties of the top-level access function not covered
 by the generic 'roundtrip' testcases.  This includes error cases.
 """
+
 from pathlib import Path
 from typing import List
 
diff --git a/tests/unit/utils/compare_nc_datasets/test_dataset_differences__additional.py b/tests/unit/utils/compare_nc_datasets/test_dataset_differences__additional.py
@@ -6,6 +6,7 @@
   * ( ALSO: "mainfunctions" (q.v.) cover the core functionality
     -- which elements are compared and what errors this constructs. )
 """
+
 import shutil
 import warnings
 
diff --git a/tests/unit/utils/compare_nc_datasets/test_dataset_differences__mainfunctions.py b/tests/unit/utils/compare_nc_datasets/test_dataset_differences__mainfunctions.py
@@ -6,6 +6,7 @@
     * ( ALSO: "additional" tests (q.v.) cover subsidiary routines and the
         main API usage modes. )
 """
+
 import numpy as np
 import pytest
 
diff --git a/tests/unit/utils/test_ncdata_copy.py b/tests/unit/utils/test_ncdata_copy.py
@@ -2,6 +2,7 @@
 
 This is generic utility function version of the copy operation.
 """
+
 import numpy as np
 import pytest
 
diff --git a/tests/unit/utils/test_save_errors.py b/tests/unit/utils/test_save_errors.py
@@ -1,6 +1,7 @@
 """
 Tests for :mod:`ncdata.utils.save_errors`
 """
+
 import re
 
 import numpy as np
diff --git a/tests/unit/xarray/test_from_xarray.py b/tests/unit/xarray/test_from_xarray.py
diff --git a/tests/unit/xarray/test_to_xarray.py b/tests/unit/xarray/test_to_xarray.py

Original file line number	Diff line number	Diff line change
`@@ -4,6 +4,7 @@`
`4`	`4`	Convert :class:`~ncdata.NcData`\s to and from Iris :class:`~iris.cube.Cube`\s.
`5`	`5`
`6`	`6`	`"""`
	`7`	`+`
`7`	`8`	`from typing import Any, AnyStr, Dict, Iterable, List, Union`
`8`	`9`
`9`	`10`	`#`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`"""General user utility functions."""`
	`2`	`+`
`2`	`3`	`from ._compare_nc_datasets import dataset_differences, variable_differences`
`3`	`4`	`from ._copy import ncdata_copy`
`4`	`5`	`from ._save_errors import save_errors`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`"""User utility routines for ncdata."""`
	`2`	`+`
`2`	`3`	`from typing import Dict, List, Union`
`3`	`4`
`4`	`5`	`import netCDF4 as nc`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	"""Tests for :mod:`ncdata`."""
	`2`	`+`
`2`	`3`	`from pathlib import Path`
`3`	`4`
`4`	`5`	`testdata_dir = Path(__file__).parent / "testdata"`