Ethanfel/ComfyUI-SelVA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: treat empty python_env as auto-managed venv trigger

Browse Source 
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 <noreply@anthropic.com>
This commit is contained in:
Ethanfel
2026-03-27 20:21:16 +01:00
parent 829f398ed0
commit a6d584bd34
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
nodes/feature_extractor.py
+2 -2
View File
@@ -133,8 +133,8 @@ class PrismAudioFeatureExtractor:
CATEGORY = PRISMAUDIO_CATEGORY CATEGORY = PRISMAUDIO_CATEGORY
def extract_features(self, video, caption_cot, fps=30.0, python_env="python", cache_dir="", synchformer_ckpt=""): 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 # Resolve python binary — auto-install managed venv if empty or default
if python_env == "python": if not python_env.strip() or python_env.strip() == "python":
python_env = _ensure_extract_env() python_env = _ensure_extract_env()
# Determine cache directory # Determine cache directory