Repository navigation
Conversation
questions and per-question runs behind the pilot's numbers, and kudos-pilot-compare.ts can recheck saved runs without the local Typesense.
…pages - benchmarks/benchlib and benchmarks/README.md come from #126, unchanged. benchmarks/kudos-pilot splits into engines, embeddings and indexes, each with a report.py that rebuilds its report from results/; engines and embeddings also keep their reference, per-side verdicts and run records. - New embedding models: gte-multilingual-base, pplx-embed-v1-0.6b, F2LLM-v2-0.6B, harrier-oss-v1-0.6b and text-embedding-3-small, plus Qwen3-Embedding-8B, 4B and Arctic shortened from their stored vectors (shorten.py). - Search steps: setup_models.py writes genki's search without the planner cut down to one step at a time (vector or fusion search, with and without ColBERT, and the whole search without ColBERT). chain_steps.sh runs them for text-embedding-3-large and pplx-embed, and the embeddings report shows which step loses the pages: the keyword fusion costs the most page MRR. - kudos-pilot-compare.ts writes per-question verdicts for the reports. The old flat results/ and its scripts go, and __pycache__ is no longer tracked. Part of #29, from the spike in #92.
5 tasks
4 of 6 tasks
This branch has not been deployed
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
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.
What and why
The Kudos pilot from #92 as a draft, so its code and the evidence behind ADR 0040 in #125 can be read and linked. It is the spike and is not for merge as it stands: code that matches the ADR comes in its own PRs. It is stacked on #91 (the Marker service), so the diff shows only the pilot.
What the benchmarks found, each from its
results/report.md:The full write-up is in a comment on #29.
How it was tested
npm run format:check,npm run lint,npm run typecheckandnpm run boundariespass;npx vitest run packages/evalpasses (70 tests).npm test: 4,713 tests pass and one fails, the console's revisjon test inapps/console/src/app.test.tsx; this PR doesn't changeapps/console, and the test passes on a branch with newer main.Each benchmark's report rebuilds from its
results/alone (report.pyin engines, embeddings and indexes).python -m unittest test_reportin engines and embeddings passes: the report refuses verdicts graded on an older reference, and every side is graded on every question of each view.benchlib's own tests pass.The runs themselves need the local pilot setup (two Postgres containers, a local Typesense copy, Marker and the embedding servers);
benchmarks/kudos-pilot/README.mdhas the commands.npm run checkpasses (all but that one console test)Checklist
packages/providers-*docs/architecture/modulesare updated if a contract, boundary or module changed (ADR 0022 is amended here; ADR 0040 in Propose the knowledge database #125 replaces the design)Merge danger
Door: two-way while it stays a draft. Merging it would bring the spike's ingest and storage changes into main, which ADR 0040 replaces.
Blast radius: none until merged.