Skip to content

eval: export the LoRA cells as a frozen-retrieval benchmark - #165

Merged
ASuresh0524 merged 6 commits into
mainfrom
eval/dump-retrieval-bench
Oct 1, 2026
Merged

ASuresh0524 merged 6 commits into
mainfrom
eval/dump-retrieval-bench

Conversation

@andylizf

Copy link
Copy Markdown
Contributor

Adds a retrieval-only mode to the eval harness and uses it to export the LoRA cells of Table 1 as a benchmark with frozen retrieval (questions plus their top-5 tiles), so a reader model can be scored without running a search serve.

  • run_bench.py --dump-retrieval DIR loads examples and builds the retriever exactly as a normal run, then writes records.jsonl, the retrieved tiles and any query images instead of calling a reader. It is resumable by example id and logs one start and one end line per example.
  • dump_bench.sh <bench> <out_root> runs it with the example sets, query images, instruction and nprobe of the LoRA cells in reproduce.sh.
  • pack_bench.py packs the dumps into one Parquet config per bench with the images embedded. The original_data column is what lib.grader reads, so a model's answers can be graded after adding final_response.
  • A hit whose tile is missing on the serve's disk is recorded in missing_tiles.jsonl instead of failing the example. pack_bench.py refuses to pack while any such tile is unfilled, unless it is listed with --allow-missing.
  • NQ-Tables repeats one example id, and the retriever cache concatenated both copies' hits. The dump keeps the first top-k.

Checked on the full export: 4,117 questions across nq, nq_tables, simpleqa, mmsearch and encyclopedic_vqa (landmarks). Grading gold answers as predictions through lib.grader nq scores 1000/1000.

@vercel

vercel Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
web Ready Ready Preview Oct 1, 2026 5:36am UTC

`Python lint` fails on this branch. Two things, both mechanical:

- `pack_bench.py` carries `# noqa: E402` on the post-`sys.path` import, but
  E402 is already in the repo-wide ruff ignore list (pyproject.toml), so the
  suppression suppresses nothing and RUF100 reports it unused.
- `ruff format` reflows two of the new files.

No behaviour change: both files still compile and the 28 existing eval tests
pass. Also merges current main into the branch.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@ASuresh0524
ASuresh0524 merged commit c4a0f53 into main Oct 1, 2026
6 checks passed

This branch was successfully deployed

1 active deployment
Preview — ae501cbf Deployed Oct 1, 2026 by vercel[bot]
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.

2 participants