Update precommit hooks (#274)

Changes:

* Manually updates the pre-commit hooks in finch

Author: Zeitsperre

.pre-commit-config.yaml

@@ -19,7 +19,7 @@ repos:
         args: [ '--allow-multiple-documents' ]
       - id: debug-statements
       - id: pretty-format-json
-        args: ['--autofix', '--no-ensure-ascii', '--no-sort-keys']
+        args: [ '--autofix', '--no-ensure-ascii', '--no-sort-keys' ]
         exclude: ".geojson"
   - repo: https://github.com/pycqa/flake8
     rev: 6.0.0
@@ -28,17 +28,17 @@ repos:
         additional_dependencies: [ 'flake8-rst-docstrings' ]
         args: [ '--config=setup.cfg' ]
   - repo: https://github.com/psf/black
-    rev: 22.12.0
+    rev: 23.1.0
     hooks:
       - id: black
         args: [ '--target-version=py38' ]
   - repo: https://github.com/PyCQA/isort
-    rev: 5.11.4
+    rev: 5.12.0
     hooks:
       - id: isort
         args: [ '--settings-file=setup.cfg' ]
   - repo: https://github.com/nbQA-dev/nbQA
-    rev: 1.6.0
+    rev: 1.6.3
     hooks:
       - id: nbqa-black
         args: [ '--target-version=py38' ]
@@ -53,17 +53,17 @@ repos:
         files: ".ipynb"
         args: [ '--keep-output', '--keep-count' ]
   - repo: https://github.com/pycqa/pydocstyle
-    rev: 6.1.1
+    rev: 6.3.0
     hooks:
       - id: pydocstyle
         args: [ '--config=setup.cfg' ]
   - repo: https://github.com/keewis/blackdoc
     rev: v0.3.8
     hooks:
       - id: blackdoc
-        additional_dependencies: [ 'black==22.8.0' ]
+        additional_dependencies: [ 'black==23.1.0' ]
   - repo: https://github.com/adrienverge/yamllint.git
-    rev: v1.28.0
+    rev: v1.29.0
     hooks:
       - id: yamllint
         args: [ '--config-file=.yamllint.yaml' ]
@@ -79,6 +79,6 @@ ci:
   autofix_prs: true
   autoupdate_branch: ''
   autoupdate_commit_msg: '[pre-commit.ci] pre-commit autoupdate'
-  autoupdate_schedule: quarterly
+  autoupdate_schedule: monthly
   skip: [ nbstripout ]
   submodules: false

docs/source/notebooks/finch-usage.ipynb

@@ -12,14 +12,7 @@
   {
    "cell_type": "code",
    "execution_count": 1,
-   "metadata": {
-    "execution": {
-     "iopub.execute_input": "2023-02-15T22:12:44.628783Z",
-     "iopub.status.busy": "2023-02-15T22:12:44.628385Z",
-     "iopub.status.idle": "2023-02-15T22:12:49.311470Z",
-     "shell.execute_reply": "2023-02-15T22:12:49.310505Z"
-    }
-   },
+   "metadata": {},
    "outputs": [],
    "source": [
     "import os\n",
@@ -44,14 +37,7 @@
   {
    "cell_type": "code",
    "execution_count": 2,
-   "metadata": {
-    "execution": {
-     "iopub.execute_input": "2023-02-15T22:12:49.317558Z",
-     "iopub.status.busy": "2023-02-15T22:12:49.317052Z",
-     "iopub.status.idle": "2023-02-15T22:12:49.322861Z",
-     "shell.execute_reply": "2023-02-15T22:12:49.321899Z"
-    }
-   },
+   "metadata": {},
    "outputs": [
     {
      "name": "stdout",
@@ -123,14 +109,7 @@
   {
    "cell_type": "code",
    "execution_count": 3,
-   "metadata": {
-    "execution": {
-     "iopub.execute_input": "2023-02-15T22:12:49.381966Z",
-     "iopub.status.busy": "2023-02-15T22:12:49.381176Z",
-     "iopub.status.idle": "2023-02-15T22:12:51.960351Z",
-     "shell.execute_reply": "2023-02-15T22:12:51.959248Z"
-    }
-   },
+   "metadata": {},
    "outputs": [],
    "source": [
     "tasmin = \"https://pavics.ouranos.ca/twitcher/ows/proxy/thredds/dodsC/birdhouse/testdata/flyingpigeon/cmip3/tasmin.sresa2.miub_echo_g.run1.atm.da.nc\"\n",
@@ -140,14 +119,7 @@
   {
    "cell_type": "code",
    "execution_count": 4,
-   "metadata": {
-    "execution": {
-     "iopub.execute_input": "2023-02-15T22:12:51.966098Z",
-     "iopub.status.busy": "2023-02-15T22:12:51.965064Z",
-     "iopub.status.idle": "2023-02-15T22:12:51.971657Z",
-     "shell.execute_reply": "2023-02-15T22:12:51.970669Z"
-    }
-   },
+   "metadata": {},
    "outputs": [
     {
      "name": "stdout",
@@ -174,14 +146,7 @@
   {
    "cell_type": "code",
    "execution_count": 5,
-   "metadata": {
-    "execution": {
-     "iopub.execute_input": "2023-02-15T22:12:51.976151Z",
-     "iopub.status.busy": "2023-02-15T22:12:51.975501Z",
-     "iopub.status.idle": "2023-02-15T22:12:52.165682Z",
-     "shell.execute_reply": "2023-02-15T22:12:52.164642Z"
-    }
-   },
+   "metadata": {},
    "outputs": [
     {
      "data": {
@@ -689,14 +654,7 @@
   {
    "cell_type": "code",
    "execution_count": 6,
-   "metadata": {
-    "execution": {
-     "iopub.execute_input": "2023-02-15T22:12:52.171175Z",
-     "iopub.status.busy": "2023-02-15T22:12:52.170318Z",
-     "iopub.status.idle": "2023-02-15T22:12:52.539761Z",
-     "shell.execute_reply": "2023-02-15T22:12:52.538731Z"
-    }
-   },
+   "metadata": {},
    "outputs": [
     {
      "name": "stdout",

finch/processes/utils.py

@@ -217,7 +217,6 @@ def compute_indices(
         input = input_queue[0]
 
         if isinstance(input, ComplexInput):
-
             if input.supported_formats[0] == FORMATS.JSON:
                 kwds[name] = json.loads(input.data)
 
@@ -698,7 +697,6 @@ def zip_files(
 
         n_files = len(all_files)
         for n, filename in enumerate(all_files):
-
             percentage = int(n / n_files * 100)
             message = f"Zipping file {n + 1} of {n_files}"
             log_function(message, percentage)

finch/processes/wps_xsubset_bbox_dataset.py

@@ -61,7 +61,6 @@ def __init__(self):
         }
 
     def _handler(self, request: WPSRequest, response: ExecuteResponse):
-
         convert_to_csv = request.inputs["output_format"][0].data == "csv"
         if not convert_to_csv:
             del self.status_percentage_steps["convert_to_csv"]

finch/processes/wps_xsubset_point_dataset.py

@@ -74,7 +74,6 @@ def __init__(self):
         }
 
     def _handler(self, request: WPSRequest, response: ExecuteResponse):
-
         convert_to_csv = request.inputs["output_format"][0].data == "csv"
         if not convert_to_csv:
             del self.status_percentage_steps["convert_to_csv"]

tests/test_wps_ensemble.py

@@ -227,7 +227,9 @@ def test_ensemble_heatwave_frequency_bbox_csv(client):
     lines = csv.split("\n")
     assert lines[0].startswith("lat,lon,time")
     n_data_rows = len(lines) - 2
-    assert n_data_rows == 2 * 2 * 4  # lat=2, lon=2, time=4 (last month is NaN, but kept in CSV)
+    assert (
+        n_data_rows == 2 * 2 * 4
+    )  # lat=2, lon=2, time=4 (last month is NaN, but kept in CSV)
 
 
 def test_ensemble_heatwave_frequency_grid_point_dates(client):

tests/test_wps_xclim_indices.py

@@ -245,7 +245,6 @@ class TestFitProcess:
     identifier = "fit"
 
     def test_simple(self, client, netcdf_datasets):
-
         inputs = [
             wps_input_file("da", netcdf_datasets["discharge"]),
             wps_literal_input("dist", "norm"),