Update precommit

Author: ghiggi

.pre-commit-config.yaml

@@ -1,7 +1,7 @@
 ---
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.6.0
+    rev: v5.0.0
     hooks:
       - id: trailing-whitespace
       - id: end-of-file-fixer
@@ -12,12 +12,12 @@ repos:
       - id: check-ast
       - id: check-added-large-files
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.6.2
+    rev: v0.9.7
     hooks:
       - id: ruff
         args: [--fix]
   - repo: https://github.com/psf/black
-    rev: 24.8.0
+    rev: 25.1.0
     hooks:
       - id: black
         language_version: python3
@@ -32,13 +32,13 @@ repos:
       - id: prettier
         types_or: [yaml, html, css, scss, javascript, json] # markdown to avoid conflicts with mdformat
   - repo: https://github.com/codespell-project/codespell
-    rev: v2.3.0
+    rev: v2.4.1
     hooks:
       - id: codespell
         types_or: [python, markdown, rst]
         additional_dependencies: [tomli]
   - repo: https://github.com/asottile/pyupgrade
-    rev: v3.17.0
+    rev: v3.19.1
     hooks:
       - id: pyupgrade
   - repo: https://github.com/MarcoGorelli/madforhooks
@@ -47,18 +47,18 @@ repos:
       # - id: conda-env-sorter # conflicts with prettier
       - id: check-execution-order
   - repo: https://github.com/executablebooks/mdformat
-    rev: 0.7.17
+    rev: 0.7.22
     hooks:
       - id: mdformat
         additional_dependencies: [mdformat-gfm, mdformat-black]
         exclude: ^.github/ # this avoid the hook to wrongly reformats md issue template files
   - repo: https://github.com/kynan/nbstripout
-    rev: 0.7.1
+    rev: 0.8.1
     hooks:
       - id: nbstripout
         args: [--keep-output]
   - repo: https://github.com/nbQA-dev/nbQA
-    rev: 1.8.7
+    rev: 1.9.1
     hooks:
       - id: nbqa-black
       - id: nbqa-ruff

CODE_OF_CONDUCT.md

@@ -58,7 +58,7 @@ representative at an online or offline event.
 
 Instances of abusive, harassing, or otherwise unacceptable behavior may be
 reported to the community leaders responsible for enforcement at
-\[INSERT CONTACT METHOD\].
+[INSERT CONTACT METHOD].
 All complaints will be reviewed and investigated promptly and fairly.
 
 All community leaders are obligated to respect the privacy and security of the

gpm/accessor/__init__.py

@@ -27,4 +27,4 @@
 """This directory defines GPM-API xarray accessors."""
 from .methods import GPM_DataArray_Accessor, GPM_Dataset_Accessor
 
-__all__ = ["GPM_Dataset_Accessor", "GPM_DataArray_Accessor"]
+__all__ = ["GPM_DataArray_Accessor", "GPM_Dataset_Accessor"]

gpm/bucket/__init__.py

@@ -46,8 +46,8 @@
 __all__ = [
     "LonLatPartitioning",
     "TilePartitioning",
-    "read",
     "merge_granule_buckets",
-    "write_granules_bucket",
+    "read",
     "write_bucket",
+    "write_granules_bucket",
 ]

gpm/bucket/writers.py

@@ -69,10 +69,10 @@ def _convert_size_to_bytes(size_str):
         "yb": 1024**8,
     }
 
-    for suffix in multipliers:
+    for suffix, multiplier in multipliers.items():
         size_str = size_str.lower().strip().strip("s")
         if size_str.lower().endswith(suffix):
-            return int(float(size_str[0 : -len(suffix)]) * multipliers[suffix])
+            return int(float(size_str[0 : -len(suffix)]) * multiplier)
     if size_str.endswith("b"):
         size_str = size_str[0:-1]
     elif size_str.endswith("byte"):

gpm/gv/__init__.py

@@ -36,11 +36,11 @@
 from gpm.gv.routines import volume_matching, retrieve_ds_sr
 
 __all__ = [
+    "calibration_summary",
     "compare_maps",
+    "reflectivity_distribution",
     "reflectivity_scatterplot",
     "reflectivity_scatterplots",
-    "reflectivity_distribution",
-    "calibration_summary",
     "retrieve_ds_sr",
     "volume_matching",
 ]

gpm/io/download.py

@@ -991,7 +991,7 @@ def download_archive(
     list_status = []
     list_versions = []
     for i, date in enumerate(dates):
-        warn_missing_files = not (i == 0 or i == len(dates) - 1 and date == end_time)
+        warn_missing_files = not (i == 0 or (i == len(dates) - 1 and date == end_time))
 
         status, available_version = _download_daily_data(
             date=date,

gpm/tests/test_utils/test_geospatial.py

@@ -300,7 +300,7 @@ def test_get_country_extent(
         get_country_extent(country)
 
     # Test typo in country name
-    country = "Afganistan"
+    country = "Afghanistan"
     with pytest.raises(ValueError) as exception_info:
         get_country_extent(country)
 

gpm/tests/test_visualization/utils.py

@@ -81,22 +81,22 @@ def save_and_check_figure(
         )
 
     # Save current figure to temporary file
-    tmp_file = tempfile.NamedTemporaryFile(suffix=image_extension, delete=False)
-    figure.savefig(tmp_file.name)
-
-    # Compare reference and temporary file
-    reference = mpl_image.imread(reference_path)
-    tmp = mpl_image.imread(tmp_file.name)
-
-    mse = np.mean((reference - tmp) ** 2)
-    assert (
-        mse < mse_tolerance
-    ), f"Figure {tmp_file.name} is not the same as {name}{image_extension}. MSE {mse} > {mse_tolerance}"
-
-    # Remove temporary file if comparison was successful
-    tmp_file.close()
-    os.remove(tmp_file.name)
-    plt.close()
+    with tempfile.NamedTemporaryFile(suffix=image_extension, delete=False) as tmp_file:
+        figure.savefig(tmp_file.name)
+
+        # Compare reference and temporary file
+        reference = mpl_image.imread(reference_path)
+        tmp = mpl_image.imread(tmp_file.name)
+
+        mse = np.mean((reference - tmp) ** 2)
+        assert (
+            mse < mse_tolerance
+        ), f"Figure {tmp_file.name} is not the same as {name}{image_extension}. MSE {mse} > {mse_tolerance}"
+
+        # Remove temporary file if comparison was successful
+        tmp_file.close()
+        os.remove(tmp_file.name)
+        plt.close()
 
 
 def get_test_name() -> str:

gpm/utils/manipulations.py

@@ -178,7 +178,7 @@ def crop_around_valid_data(xr_obj, variable=None):
     da = get_xarray_variable(xr_obj, variable=variable)
 
     # Create a boolean mask indicating where da is not NaN
-    valid_mask = da.notnull().compute()  # noqa: PD004
+    valid_mask = da.notnull().compute()
 
     # Raise error if not valid data
     if not np.any(valid_mask).item():