PR #771 code style cleanup

using `darker -v -r master openeo`

Author: soxofaan

openeo/metadata.py

@@ -519,7 +519,7 @@ class CollectionMetadata(CubeMetadata):
 
     """
 
-    def __init__(self, metadata: dict, dimensions: List[Dimension] = None, federation = None):
+    def __init__(self, metadata: dict, dimensions: List[Dimension] = None, federation=None):
         self._orig_metadata = metadata
         self._federation = federation
         if dimensions is None:
@@ -641,7 +641,7 @@ def extent(self) -> dict:
         return self._orig_metadata.get("extent")
 
     def _repr_html_(self):
-        return render_component("collection", data=self._orig_metadata, parameters={'federation': self._federation})
+        return render_component("collection", data=self._orig_metadata, parameters={"federation": self._federation})
 
     def __str__(self) -> str:
         bands = self.band_names if self.has_band_dimension() else "no bands dimension"

openeo/rest/connection.py

@@ -777,10 +777,11 @@ def list_file_formats(self) -> dict:
             load=lambda: self.get('/file_formats', expected_status=200).json()
         )
         federation = self.capabilities().ext_federation_backend_details()
-        return VisualDict("file-formats", data=formats, parameters={
-            'missing': formats.get("federation:missing", None),
-            'federation': federation
-        })
+        return VisualDict(
+            "file-formats",
+            data=formats,
+            parameters={"missing": formats.get("federation:missing", None), "federation": federation},
+        )
 
     def list_service_types(self) -> dict:
         """
@@ -805,7 +806,7 @@ def list_udf_runtimes(self) -> dict:
             load=lambda: self.get('/udf_runtimes', expected_status=200).json()
         )
         federation = self.capabilities().ext_federation_backend_details()
-        return VisualDict("udf-runtimes", data=runtimes, parameters={'federation': federation})
+        return VisualDict("udf-runtimes", data=runtimes, parameters={"federation": federation})
 
     def list_services(self) -> dict:
         """
@@ -816,11 +817,15 @@ def list_services(self) -> dict:
         # TODO return parsed service objects
         services = self.get('/services', expected_status=200).json()["services"]
         federation = self.capabilities().ext_federation_backend_details()
-        return VisualList("data-table", data=services, parameters={
-            'columns': 'services',
-            'missing': services.get("federation:missing", None),
-            'federation': federation
-        })
+        return VisualList(
+            "data-table",
+            data=services,
+            parameters={
+                "columns": "services",
+                "missing": services.get("federation:missing", None),
+                "federation": federation,
+            },
+        )
 
     def describe_collection(self, collection_id: str) -> dict:
         """
@@ -838,7 +843,7 @@ def describe_collection(self, collection_id: str) -> dict:
         # TODO: add caching #383
         data = self.get(f"/collections/{collection_id}", expected_status=200).json()
         federation = self.capabilities().ext_federation_backend_details()
-        return VisualDict("collection", data=data, parameters={'federation': federation})
+        return VisualDict("collection", data=data, parameters={"federation": federation})
 
     def collection_items(
         self,
@@ -877,7 +882,16 @@ def collection_items(
             params['limit'] = limit
 
         federation = self.capabilities().ext_federation_backend_details()
-        return paginate(self, url, params, lambda response, page: VisualDict("items", data = response, parameters = {'show-map': True, 'heading': 'Page {} - Items'.format(page), 'federation': federation}))
+        return paginate(
+            self,
+            url,
+            params,
+            lambda response, page: VisualDict(
+                "items",
+                data=response,
+                parameters={"show-map": True, "heading": "Page {} - Items".format(page), "federation": federation},
+            ),
+        )
 
     def collection_metadata(self, name) -> CollectionMetadata:
         # TODO: duplication with `Connection.describe_collection`: deprecate one or the other?
@@ -920,11 +934,11 @@ def describe_process(self, id: str, namespace: Optional[str] = None) -> dict:
         federation = self.capabilities().ext_federation_backend_details()
         for process in processes:
             if process["id"] == id:
-                return VisualDict("process", data=process, parameters={
-                    'show-graph': True,
-                    'provide-download': False,
-                    'federation': federation
-                })
+                return VisualDict(
+                    "process",
+                    data=process,
+                    parameters={"show-graph": True, "provide-download": False, "federation": federation},
+                )
 
         raise OpenEoClientException("Process does not exist.")
 
@@ -1514,14 +1528,18 @@ def list_files(self) -> List[UserFile]:
 
         :return: List of the user-uploaded files.
         """
-        data = self.get('/files', expected_status=200).json()
-        files = [UserFile.from_metadata(metadata=f, connection=self) for f in data.get('files', [])]
+        data = self.get("/files", expected_status=200).json()
+        files = [UserFile.from_metadata(metadata=f, connection=self) for f in data.get("files", [])]
         federation = self.capabilities().ext_federation_backend_details()
-        return VisualList("data-table", data=files, parameters={
-            'columns': 'files',
-            'missing': data.get('federation:missing', None),
-            'federation': federation,
-        })
+        return VisualList(
+            "data-table",
+            data=files,
+            parameters={
+                "columns": "files",
+                "missing": data.get("federation:missing", None),
+                "federation": federation,
+            },
+        )
 
     def get_file(
         self, path: Union[str, PurePosixPath], metadata: Optional[dict] = None

openeo/rest/job.py

@@ -82,10 +82,14 @@ def __repr__(self):
     def _repr_html_(self):
         data = self.describe()
         capabilities = self.connection.capabilities()
-        return render_component('job', data=data, parameters={
-            'currency': capabilities.currency(),
-            'federation': capabilities.ext_federation_backend_details(),
-        })
+        return render_component(
+            "job",
+            data=data,
+            parameters={
+                "currency": capabilities.currency(),
+                "federation": capabilities.ext_federation_backend_details(),
+            },
+        )
 
     @openeo_endpoint("GET /jobs/{job_id}")
     def describe(self) -> dict:

openeo/rest/models/general.py

@@ -52,7 +52,7 @@ class CollectionListingResponse(list):
 
     __slots__ = ["_data", "_connection"]
 
-    def __init__(self, response_data: dict, connection = None):
+    def __init__(self, response_data: dict, connection=None):
         self._data = response_data
         self._connection = connection
         # Mimic original list of collection metadata dictionaries
@@ -68,7 +68,7 @@ def _repr_html_(self):
             parameters={
                 "missing": self.ext_federation_missing(),
                 "federation": federation,
-            }
+            },
         )
 
     @property
@@ -113,7 +113,7 @@ class ProcessListingResponse(list):
 
     __slots__ = ["_data", "_connection"]
 
-    def __init__(self, response_data: dict, connection = None):
+    def __init__(self, response_data: dict, connection=None):
         self._data = response_data
         self._connection = connection
         # Mimic original list of process metadata dictionaries
@@ -131,7 +131,7 @@ def _repr_html_(self):
                 "provide-download": False,
                 "missing": self.ext_federation_missing(),
                 "federation": federation,
-            }
+            },
         )
 
     @property
@@ -177,7 +177,7 @@ class JobListingResponse(list):
 
     __slots__ = ["_data", "_connection"]
 
-    def __init__(self, response_data: dict, connection = None):
+    def __init__(self, response_data: dict, connection=None):
         self._data = response_data
         self._connection = connection
         # Mimic original list of process metadata dictionaries
@@ -194,7 +194,7 @@ def _repr_html_(self):
                 "columns": "jobs",
                 "missing": self.ext_federation_missing(),
                 "federation": federation,
-            }
+            },
         )
 
     @property
@@ -243,7 +243,7 @@ class LogsResponse(list):
 
     __slots__ = ["_data", "_connection"]
 
-    def __init__(self, response_data: dict, *, log_level: Optional[str] = None, connection = None):
+    def __init__(self, response_data: dict, *, log_level: Optional[str] = None, connection=None):
         self._data = response_data
         self._connection = connection
 
@@ -277,7 +277,7 @@ def _repr_html_(self):
             parameters={
                 "missing": self.ext_federation_missing(),
                 "federation": federation,
-            }
+            },
         )
 
     @property

openeo/rest/service.py

@@ -26,10 +26,14 @@ def __repr__(self):
     def _repr_html_(self):
         data = self.describe_service()
         capabilities = self.connection.capabilities()
-        return VisualDict('service', data = data, parameters = {
-            'currency': capabilities.currency(),
-            'federation': capabilities.ext_federation_backend_details(),
-        })
+        return VisualDict(
+            "service",
+            data=data,
+            parameters={
+                "currency": capabilities.currency(),
+                "federation": capabilities.ext_federation_backend_details(),
+            },
+        )
 
     def describe_service(self):
         """ Get all information about a secondary web service."""

openeo/rest/udp.py

@@ -90,11 +90,15 @@ def __init__(self, user_defined_process_id: str, connection: Connection):
     def _repr_html_(self):
         process = self.describe()
         federation = self._connection.capabilities().ext_federation_backend_details()
-        return render_component('process', data=process, parameters = {
-            'show-graph': True,
-            'provide-download': False,
-            'federation': federation,
-        })
+        return render_component(
+            "process",
+            data=process,
+            parameters={
+                "show-graph": True,
+                "provide-download": False,
+                "federation": federation,
+            },
+        )
 
     def store(
         self,