minor code formatting cleanups

mix of `ruff format` and some manual fixes

Author: soxofaan

openeo/metadata.py

@@ -46,7 +46,7 @@ def __init__(self, type: str, name: str):
     def __repr__(self):
         return "{c}({f})".format(
             c=self.__class__.__name__,
-            f=", ".join("{k!s}={v!r}".format(k=k, v=v) for (k, v) in self.__dict__.items())
+            f=", ".join("{k!s}={v!r}".format(k=k, v=v) for (k, v) in self.__dict__.items()),
         )
 
     def __eq__(self, other):
@@ -201,7 +201,7 @@ def filter_bands(self, bands: List[Union[int, str]]) -> BandDimension:
         """
         return BandDimension(
             name=self.name,
-            bands=[self.bands[self.band_index(b)] for b in bands]
+            bands=[self.bands[self.band_index(b)] for b in bands],
         )
 
     def append_band(self, band: Union[Band, str]) -> BandDimension:
@@ -213,7 +213,7 @@ def append_band(self, band: Union[Band, str]) -> BandDimension:
 
         return BandDimension(
             name=self.name,
-            bands=self.bands + [band]
+            bands=self.bands + [band],
         )
 
     def rename_labels(self, target, source) -> Dimension:
@@ -994,7 +994,7 @@ def bands_from_stac_collection(
         elif _PYSTAC_1_12_ITEM_ASSETS and collection.item_assets:
             return self._bands_from_item_assets(collection.item_assets)
         elif (
-             collection.ext.has("item_assets")
+            collection.ext.has("item_assets")
             and collection.extra_fields.get("item-assets")
             and collection.ext.item_assets
         ):

openeo/rest/_datacube.py

@@ -88,7 +88,7 @@ def _build_pgnode(
         process_id: str,
         arguments: Optional[dict] = None,
         namespace: Optional[str] = None,
-        **kwargs
+        **kwargs,
     ) -> PGNode:
         """
         Helper to build a PGNode from given argument dict and/or kwargs,
@@ -213,9 +213,7 @@ def from_file(
         """
         path = pathlib.Path(path)
         code = path.read_text(encoding="utf-8")
-        return cls(
-            code=code, runtime=runtime, version=version, context=context, _source=path
-        )
+        return cls(code=code, runtime=runtime, version=version, context=context, _source=path)
 
     @classmethod
     def from_url(
@@ -239,9 +237,7 @@ def from_url(
         resp = requests.get(url)
         resp.raise_for_status()
         code = resp.text
-        return cls(
-            code=code, runtime=runtime, version=version, context=context, _source=url
-        )
+        return cls(code=code, runtime=runtime, version=version, context=context, _source=url)
 
     def _guess_runtime(self, connection: Optional[Connection] = None) -> str:
         """Guess UDF runtime from UDF source (path) or source code."""
@@ -250,9 +246,7 @@ def _guess_runtime(self, connection: Optional[Connection] = None) -> str:
         if isinstance(self._source, pathlib.Path):
             language = self._guess_runtime_from_suffix(self._source.suffix)
         elif isinstance(self._source, str):
-            url_match = re.match(
-                r"https?://.*?(?P<suffix>\.\w+)([&#].*)?$", self._source
-            )
+            url_match = re.match(r"https?://.*?(?P<suffix>\.\w+)([&#].*)?$", self._source)
             if url_match:
                 language = self._guess_runtime_from_suffix(url_match.group("suffix"))
         if not language:
@@ -316,6 +310,7 @@ def build_child_callback(
         # Assume given reducer is a simple predefined reduce process_id
         # TODO: avoid local import (workaround for circular import issue)
         import openeo.processes
+
         if process in openeo.processes.__dict__:
             process_params = get_parameter_names(openeo.processes.__dict__[process])
             # TODO: switch to "Callable" handling here