Add comment for suppressing import error

rahimftd · rahimftd · commit 40450986be8d · 2025-09-26T18:49:40.000Z
Signed-off-by: Rahim Dharssi &lt;rahimftd@cisco.com&gt;
diff --git a/nemo_run/core/execution/skypilot_jobs.py b/nemo_run/core/execution/skypilot_jobs.py
@@ -23,6 +23,7 @@
 
     _SKYPILOT_AVAILABLE = True
 except ImportError:
+    # suppress import error so we don't crash if skypilot is not installed.
     pass
 
 logger = logging.getLogger(__name__)
diff --git a/nemo_run/run/torchx_backend/schedulers/skypilot_jobs.py b/nemo_run/run/torchx_backend/schedulers/skypilot_jobs.py
@@ -56,6 +56,7 @@
         ManagedJobStatus.FAILED_CONTROLLER: AppState.FAILED,
     }
 except ImportError:
+    # suppress import error so we don't crash if skypilot is not installed.
     pass
 
 SKYPILOT_JOB_DIRS = os.path.join(get_nemorun_home(), ".skypilot_jobs.json")

Original file line number	Diff line number	Diff line change
`@@ -56,6 +56,7 @@`
`56`	`56`	`ManagedJobStatus.FAILED_CONTROLLER: AppState.FAILED,`
`57`	`57`	`}`
`58`	`58`	`except ImportError:`
	`59`	`+ # suppress import error so we don't crash if skypilot is not installed.`
`59`	`60`	`pass`
`60`	`61`
`61`	`62`	`SKYPILOT_JOB_DIRS = os.path.join(get_nemorun_home(), ".skypilot_jobs.json")`