Skip to content

Commit 4103606

Browse files
meizhong986claude
andcommitted
fix(notebook/kaggle): qwen in pass*_quality dropdowns + translate cmd argv split (#315, #318)
#318: pass1_quality and pass2_quality #@PARAM dropdowns were missing "qwen" even though qwen is a valid --mode value in whisperjav.main argparse. Adds it to both passes for parity. Reported by jasial2. #315: cell-3's translate cmd was passing the multi-token Kaggle command (f"{sys.executable} -m whisperjav.translate.cli") as a single argv token, producing "[Errno 2] No such file or directory: '/usr/bin/python3 -m whisperjav.translate.cli'". Mirrors the split pattern already in cell-2's build_cmd(): checks endswith('whisperjav-translate') for the Colab single-executable case, else .split() for the Kaggle module form. Reported by jasial2. Both fixes ship off-release on main; the Kaggle notebook installs WhisperJAV from git+...@main, so users get the fix immediately on next notebook run. No version bump needed. Notebook source format note: NotebookEdit collapsed the affected cells' source from list-of-strings to single-string format. nbformat accepts both forms and Jupyter renders identically — no functional change, but it produces a noisy diff. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 16f49ce commit 4103606

1 file changed

Lines changed: 2 additions & 221 deletions

File tree

0 commit comments

Comments
 (0)