misc: Address some comments

Author: ZoeLeibowitz

devito/petsc/iet/passes.py

@@ -10,7 +10,7 @@
 from devito.types import Symbol, Scalar
 from devito.types.basic import DataSymbol
 from devito.tools import frozendict
-import devito.logger as dl
+import devito.logger
 
 from devito.petsc.types import (PetscMPIInt, PetscErrorCode, MultipleFieldData,
                                 PointerIS, Mat, CallbackVec, Vec, CallbackMat, SNES,
@@ -201,7 +201,7 @@ def solve(self):
 
     @cached_property
     def logger(self):
-        log_level = dl.logger.level
+        log_level = devito.logger.logger.level
         return PetscLogger(
             log_level, self.get_info, **self.common_kwargs
         )

devito/petsc/logging.py

@@ -192,9 +192,11 @@ def get_val(val):
         # - If the function returns multiple values (e.g., KSPGetTolerances),
         #   return a dictionary mapping each output name to its value,
         #   e.g., {'rtol': val0, 'atol': val1, ...}.
-        if len(obj_mapper) > 1:
-            return {k: get_val(v) for k, v in obj_mapper.items()}
-        return get_val(next(iter(obj_mapper.values())))
+        info = {k: get_val(v) for k, v in obj_mapper.items()}
+        if len(info) == 1:
+            return info.popitem()[1]
+        else:
+            return info
 
 
 @dataclass

devito/petsc/solver_parameters.py

@@ -38,4 +38,4 @@ def format_options_prefix(options_prefix):
 
     if options_prefix.endswith("_"):
         return options_prefix
-    return options_prefix + "_"
+    return f"{options_prefix}_"

requirements-optional.txt

@@ -3,4 +3,4 @@ pillow>11,<11.3.1
 pyrevolve==2.2.6
 scipy<1.15.4
 distributed<2025.7.1
-click<9.0
+click<9.0

requirements.txt

@@ -9,4 +9,4 @@ multidict<6.3
 anytree>=2.4.3,<=2.13.0
 cloudpickle<3.1.2
 packaging<25.1
-petsctools==2025.1
+petsctools<=2025.1