Skip to content

[BCI] QVAC-17072 chore: prepare for registry-based whisper-cpp consumption#1585

Closed
sharmaraju352 wants to merge 2 commits into
feat/bci-whispercpp-2-whisper-libfrom
feat/bci-whispercpp-3-registry
Closed

[BCI] QVAC-17072 chore: prepare for registry-based whisper-cpp consumption#1585
sharmaraju352 wants to merge 2 commits into
feat/bci-whispercpp-2-whisper-libfrom
feat/bci-whispercpp-3-registry

Conversation

@sharmaraju352

@sharmaraju352 sharmaraju352 commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Stacked PR

This is PR 3 of 6 in the BCI addon stack (based on PR 2: #1584):

  1. QVAC-17057 - POC with batch transcription ([BCI] QVAC-17057 feat: add bci-whispercpp POC with batch transcription #1583)
  2. QVAC-17071 - Update qvac-ext-lib-whisper.cpp ([BCI] QVAC-17071 chore: switch whisper-cpp overlay to tetherto fork #1584)
  3. QVAC-17072 - Update qvac-registry-vcpkg with updated whisper.cpp (this PR)
  4. QVAC-17062 - Add streaming capabilities and integration tests ([BCI] QVAC-17062 feat: add streaming transcription and integration tests #1586)
  5. QVAC-17058 - Add empty BCI workflows to main ([BCI] QVAC-17058 ci: add empty BCI workflows to enable branch triggering #1587)
  6. QVAC-17063 - Add full CI workflows for all platforms ([BCI] QVAC-17063 ci: add full CI workflows for prebuilds, desktop and mobile tests #1588)

Dependencies

Test plan

  • Update vcpkg-configuration.json baseline to registry commit hash
  • npm run build succeeds with whisper-cpp from registry (no local overlay)
  • npm run test:integration passes

…tion

Remove the whisper-cpp version override from vcpkg.json since the local
overlay portfile already pins the version. The whisper-cpp overlay is
retained until the qvac-registry-vcpkg is updated with the BCI-aware
port, at which point the overlay can be removed and the baseline bumped.

Made-with: Cursor
@sharmaraju352 sharmaraju352 force-pushed the feat/bci-whispercpp-3-registry branch from e1af991 to 18c2db2 Compare April 15, 2026 09:57
@sharmaraju352 sharmaraju352 changed the title QVAC-17072 chore[notask]: consume whisper-cpp from registry, remove local overlay QVAC-17072 chore[BCI]: prepare for registry-based whisper-cpp consumption Apr 15, 2026
@sharmaraju352 sharmaraju352 changed the title QVAC-17072 chore[BCI]: prepare for registry-based whisper-cpp consumption [BCI] QVAC-17072 chore: prepare for registry-based whisper-cpp consumption Apr 15, 2026
@sharmaraju352

Copy link
Copy Markdown
Contributor Author

Closing — this PR was a placeholder that is now superseded.

The BCI whisper.cpp changes are handled via:

The 6-PR stack has been collapsed to 4 PRs. PRs #1584 and #1585 (whisper-lib switch + registry consumption) were net-zero (commit + revert) and are no longer needed.

New stack:

  1. [BCI] QVAC-17057 feat: add bci-whispercpp POC with batch transcription #1583 — POC with batch transcription (includes all review fixes)
  2. [BCI] QVAC-17062 feat: add streaming transcription and integration tests #1586 — Streaming transcription
  3. [BCI] QVAC-17058 ci: add empty BCI workflows to enable branch triggering #1587 — Empty CI workflows
  4. [BCI] QVAC-17063 ci: add full CI workflows for prebuilds, desktop and mobile tests #1588 — Full CI workflows

@github-actions

Copy link
Copy Markdown
Contributor

Tier-based Approval Status

**PR Tier:** TIER1

**Current Status:** ❌ PENDING

**Requirements:**
- 1 Team Member approval ❌ (0/1)
- 1 Team Lead OR Management approval ❌ (0/1)



---
*This comment is automatically updated when reviews change.*

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant