Apply suggestions from code review

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Author: edyoshikun

applications/dynaclr/configs/training/slurm/train.sh

@@ -83,9 +83,7 @@ WANDB_ID_FILE="${RUN_DIR}/.wandb_run_id"
 if [ -z "${WANDB_RUN_ID:-}" ]; then
   if [ -f "${WANDB_ID_FILE}" ]; then
     WANDB_RUN_ID="$(cat "${WANDB_ID_FILE}")"
-  else
-    WANDB_RUN_ID="$(python -c 'import secrets; print(secrets.token_hex(4))')"
-    echo "${WANDB_RUN_ID}" > "${WANDB_ID_FILE}"
+    WANDB_RUN_ID="$(uv run --project "$WORKSPACE_DIR" python -c 'import secrets; print(secrets.token_hex(4))')"
   fi
 fi
 

packages/viscy-utils/src/viscy_utils/cli.py

@@ -87,7 +87,7 @@ def _configure_slurm_requeue(config: Namespace, subcommand: str | None) -> None:
     root = config[subcommand] if subcommand is not None else config
     if not isinstance(root, Namespace):
         return
-    if not root.get("slurm_auto_requeue", False):
+    if not config.get("slurm_auto_requeue", False):
         return
     if not SLURMEnvironment.detect():
         return
@@ -98,10 +98,15 @@ def _configure_slurm_requeue(config: Namespace, subcommand: str | None) -> None:
     if plugins is None:
         plugins = []
     elif not isinstance(plugins, list):
-        plugins = [plugins]
-    if any(isinstance(p, SLURMEnvironment) for p in plugins):
+    if any(
+        isinstance(p, SLURMEnvironment)
+        or (isinstance(p, Namespace) and p.get("class_path") == "lightning.pytorch.plugins.environments.SLURMEnvironment")
+        for p in plugins
+    ):
         return
-    plugins.append(SLURMEnvironment(auto_requeue=True, requeue_signal=signal.SIGUSR1))
+    plugins.append(
+        lazy_instance(SLURMEnvironment, auto_requeue=True, requeue_signal=signal.SIGUSR1.value)
+    )
     trainer["plugins"] = plugins