From 9d9638e854de86f3f2c427e1d582dc44f0c428f7 Mon Sep 17 00:00:00 2001 From: Jatin K Malik Date: Mon, 13 Jul 2026 14:48:42 -0700 Subject: [PATCH] style(whispercpp): drop long comment on default thread count (#505) Per review on #492, the rationale lives in the PR/history via git blame. --- src/vocalinux/speech_recognition/recognition_manager.py | 9 --------- 1 file changed, 9 deletions(-) diff --git a/src/vocalinux/speech_recognition/recognition_manager.py b/src/vocalinux/speech_recognition/recognition_manager.py index 46b57c8..d47f698 100644 --- a/src/vocalinux/speech_recognition/recognition_manager.py +++ b/src/vocalinux/speech_recognition/recognition_manager.py @@ -1266,15 +1266,6 @@ class SpeechRecognitionManager: elif has_gpu_libs: n_threads = max(1, multiprocessing.cpu_count() // 4) else: - # Leave a couple of cores for the OS and audio capture, and avoid - # oversubscribing hybrid CPUs. Using *every* core drags the slow - # efficiency cores into the pool; whisper.cpp synchronises all threads - # at each decode step, so the barrier stalls on the slowest core and - # throughput collapses. On an 8-core hybrid CPU (Intel Core Ultra, - # P+E cores) the old `min(cpu_count, 8)` picked 8 = every core and - # measured RTF jumped from ~0.14x (6 threads) to ~2.1x (8 threads) — - # a ~15x slowdown. Cap at 8; base/small models rarely scale past that. - # Users can still override with the `whispercpp_n_threads` setting. n_threads = max(1, min(multiprocessing.cpu_count() - 2, 8)) load_start_time = time.time()