Fix some variable names and remove a print.

Signed-off-by: Michael Johansen <[email protected]>

Author: Michael Johansen

src/nipanel/_convert.py

@@ -104,7 +104,6 @@ def _get_best_matching_type(python_value: object) -> str:
     for candidate in candidates:
         containers_str = ".".join(container_types)
         python_typename = f"{containers_str}.{candidate}" if containers_str else candidate
-        print(python_typename)
         if python_typename not in _SUPPORTED_PYTHON_TYPES:
             continue
         best_matching_type = python_typename

src/nipanel/converters/protobuf_types.py

@@ -59,14 +59,14 @@ def to_protobuf_message(self, python_value: Collection[Collection[float]]) -> Do
         if rows:
             visitor = iter(python_value)
             first_subcollection = next(visitor)
-            columns = len(first_subcollection) if rows else 0
+            columns = len(first_subcollection)
         else:
             columns = 0
-        if not all(len(sublist) == columns for sublist in python_value):
+        if not all(len(subcollection) == columns for subcollection in python_value):
             raise ValueError("All subcollections must have the same length.")
 
         # Create a flat list in row major order.
-        flat_list = [item for sublist in python_value for item in sublist]
+        flat_list = [item for subcollection in python_value for item in subcollection]
         return Double2DArray(rows=rows, columns=columns, data=flat_list)
 
     def to_python_value(self, protobuf_message: Double2DArray) -> Collection[Collection[float]]: