From a6d584bd34d915f79c01a58a250e907d7652940f Mon Sep 17 00:00:00 2001 From: Ethanfel Date: Fri, 27 Mar 2026 20:21:16 +0100 Subject: [PATCH] fix: treat empty python_env as auto-managed venv trigger Empty string from clearing the node field caused subprocess to execute '' which raises PermissionError. Now any blank or 'python' value uses the auto-installed venv. Co-Authored-By: Claude Sonnet 4.6 --- nodes/feature_extractor.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/nodes/feature_extractor.py b/nodes/feature_extractor.py index db306f7..8e36548 100644 --- a/nodes/feature_extractor.py +++ b/nodes/feature_extractor.py @@ -133,8 +133,8 @@ class PrismAudioFeatureExtractor: CATEGORY = PRISMAUDIO_CATEGORY def extract_features(self, video, caption_cot, fps=30.0, python_env="python", cache_dir="", synchformer_ckpt=""): - # Resolve python binary — auto-install managed venv if using default - if python_env == "python": + # Resolve python binary — auto-install managed venv if empty or default + if not python_env.strip() or python_env.strip() == "python": python_env = _ensure_extract_env() # Determine cache directory