Fix formatting inconsistencies and improve error message clarity

- Adjusted whitespace in print statements for better readability in the DICOM data loader operator.
- Enhanced the error message in the AppGenerator class to improve clarity regarding valid model_id formats.
- Corrected spacing in the file processing output of the generic directory scanner operator.
- Removed unused import from the test_run_command module to clean up the code.

Signed-off-by: Victor Chang <[email protected]>

Author: mocsharp

monai/deploy/operators/dicom_data_loader_operator.py

@@ -437,10 +437,10 @@ def test():
                 print(f"   'SeriesDescription': {ds.SeriesDescription if ds.SeriesDescription else ''}")
                 print(
                     "   'IssuerOfPatientID':"
-                    f" {ds.get('IssuerOfPatientID', '').repval if ds.get('IssuerOfPatientID', '') else '' }"
+                    f" {ds.get('IssuerOfPatientID', '').repval if ds.get('IssuerOfPatientID', '') else ''}"
                 )
                 try:
-                    print(f"   'IssuerOfPatientID': {ds.IssuerOfPatientID if ds.IssuerOfPatientID else '' }")
+                    print(f"   'IssuerOfPatientID': {ds.IssuerOfPatientID if ds.IssuerOfPatientID else ''}")
                 except AttributeError:
                     print(
                         "    If the IssuerOfPatientID does not exist, ds.IssuerOfPatientID would throw AttributeError."

tools/pipeline-generator/pipeline_generator/generator/app_generator.py

@@ -99,7 +99,11 @@ def generate_app(
 
         if not model_id or not re.match(model_id_pattern, model_id):
             raise ValueError(
-                f"Invalid model_id: {model_id}. Only alphanumeric characters, hyphens, underscores, and single slashes between segments are allowed. No leading/trailing slashes, consecutive slashes, or '..' allowed."
+                (
+                    f"Invalid model_id: {model_id}. Only alphanumeric characters, hyphens, "
+                    "underscores, and single slashes between segments are allowed. "
+                    "No leading/trailing slashes, consecutive slashes, or '..' allowed."
+                )
             )
 
         # Create output directory

tools/pipeline-generator/pipeline_generator/templates/operators/generic_directory_scanner_operator.py

@@ -192,7 +192,7 @@ def emit(self, data, name):
 
         # Process a few files
         for i in range(min(3, len(scanner._files))):
-            print(f"\n--- Processing file {i+1} ---")
+            print(f"\n--- Processing file {i + 1} ---")
             scanner.compute(None, mock_output, None)
 
 

tools/pipeline-generator/tests/test_run_command.py

@@ -12,7 +12,6 @@
 """Tests for the run command with validation fixes."""
 
 import subprocess
-from pathlib import Path
 from unittest.mock import Mock, patch
 
 from click.testing import CliRunner