Skip to content

fix: honor active embed model in query vector lookups#527

Open
kuishou68 wants to merge 1 commit intotobi:mainfrom
kuishou68:fix-query-embed-model-override
Open

fix: honor active embed model in query vector lookups#527
kuishou68 wants to merge 1 commit intotobi:mainfrom
kuishou68:fix-query-embed-model-override

Conversation

@kuishou68
Copy link
Copy Markdown
Contributor

Summary

  • use the active llm embed model for query-time vector lookups instead of the hardcoded default alias
  • keep hybrid and vector-only query paths aligned with the model used to build embeddings
  • add regression coverage for vector and hybrid query routing

Closes #497

Testing

  • corepack pnpm exec vitest run test/store.test.ts -t "vectorSearchQuery uses the active llm embed model for vector lookups" --reporter=dot
  • corepack pnpm exec vitest run test/store.test.ts -t "hybridQuery uses the active llm embed model for precomputed vector lookups" --reporter=dot
  • corepack pnpm build

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.

QMD query mode ignores custom QMD_EMBED_MODEL in store path and causes 768/1024 dimension mismatch

2 participants