setup --speakers: opt-in diarization install + actionable messages#23
Conversation
Speaker detection (pyannote+torch, ~2GB) is omitted from the hermetic runtime. Previously the CLI just said "not available (source install only)" with no way to enable it. Add `podcli setup --speakers` to install the stack into the hermetic Python, and make every gate point at the exact fix: - info: "run: podcli setup --speakers" / "podcli env set HF_TOKEN <token>" - the not-set-up warning distinguishes missing deps vs missing token - speaker_detection's ImportError says "run: podcli setup --speakers"
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughAdds a ChangesSpeaker Diarization Setup
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 golangci-lint (2.12.2)level=error msg="[linters_context] typechecking error: pattern ./...: directory prefix . does not contain main module or its selected dependencies" Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Speaker detection deps (pyannote+torch, ~2GB) are excluded from the hermetic runtime. Adds 'podcli setup --speakers' to install them on demand, and replaces dead-end 'source install only' messages with the exact command to run (setup --speakers / env set HF_TOKEN).
Summary by CodeRabbit
New Features
--speakersflag to setup command for installing speaker diarization dependencies.Documentation