QVAC-5558: Disabled parallel configure in whisper-cpp and llama-cpp#31
Merged
Conversation
yuranich
approved these changes
Sep 9, 2025
olek-tether
approved these changes
Sep 9, 2025
5 tasks
gianni-cor
pushed a commit
that referenced
this pull request
Jun 5, 2026
…whisper.cpp@master 128dae42 (PR #31 supertonic_optimizations merge) (#184) All three ports source the same `tetherto/qvac-ext-lib-whisper.cpp` repo (whisper root / parakeet-cpp/ subfolder / tts-cpp/ subfolder), so this moves the trio to the same master HEAD and the same archive SHA512 to keep the registry self-consistent. - whisper-cpp 1.8.5#2 -> 1.8.5#3: no whisper-root change since #2 (a34cb6da was just PR #34 KV-cache CPU-buffer fallback); this bump keeps the source archive aligned with parakeet-cpp and tts-cpp. - parakeet-cpp 2026-06-04#0 -> 2026-06-04#1: no parakeet-cpp/ change since #0 (a34cb6da was already PR #38 Adreno regex); SHA512 alignment. - tts-cpp 2026-06-03#1 -> 2026-06-05#0: picks up PR #31 -- QVAC-18605 Supertonic Vulkan/Metal optimisations (rounds 1-13, ~34x realtime on Apple M-series Metal), QVAC-19254 sched + cpu_backend refactor for Adreno OpenCL, and QVAC-19213 Adreno-generation regex parse fix. EngineOptions gains six tuning fields the downstream tts-ggml addon will start exposing in a follow-up PR (precision, f16_attn, vulkan_device, f16_weights, f16_weights_deny_list, kv_attn_type). - tts-cpp ggml-speech floor: 2026-06-02 -> 2026-06-04 (harmonises with whisper-cpp and parakeet-cpp; the QVAC-19254 Adreno OpenCL elementwise kernels live in ggml-speech 2026-06-04). Validation: - Local Metal bench post-merge: F1 92 ms / 34.07x RT, M1 95 ms / 33.63x. - TTS GGML overlay CI on tetherto/qvac (runs/26983833143 + 27006424023): 26/27 jobs passing; the one failure is the structural merge-guard "needs 'verified' label" check that fires on every workflow_dispatch run regardless of code (same failure on the pre-merge baseline run). - All 38 supertonic ctests pass post-merge. Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Mac runners seem to have an issue with one of the cmake configure steps, which tries to configure a file. As the release and debug configurations are configured in parallel, there seems to be a race condition in these ports when the file is configured, causing intermittent build failures.
By disabling parallel configure in the port, the race condition is avoided and the mac builds become more reliable.