docs(evaluation) 15/15: feature guide and a sample dataset - #826
docs(evaluation) 15/15: feature guide and a sample dataset#826Ahmath-Gadji wants to merge 1 commit into
Conversation
|
Warning Review limit reached
Next review available in: 59 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (4)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
e969682 to
58c529a
Compare
63efdf4 to
175d6d9
Compare
|
@coderabbitai full review |
✅ Action performedFull review finished. Your included review limit is currently reached under our Fair Usage Limits Policy. This review may still proceed through usage-based billing if eligible. Your next included review will be available in 56 minutes. |
58c529a to
cff68d6
Compare
175d6d9 to
1aa6deb
Compare
cff68d6 to
958387a
Compare
1aa6deb to
58caca1
Compare
958387a to
87f654b
Compare
58caca1 to
3899a03
Compare
Adds the Evaluation section to CLAUDE.md — the flow, the layering rule, why there are two promptfoo configs, and the contracts that are deliberately not configuration. `tests/evaluation/` ships a ready-made test set: 11 questions over 6 documents, every answer checked against the source rather than generated. The corpus itself is not committed (third-party PDFs, ~36 MB); `corpus.txt` lists the filenames and the README explains how to assemble and upload it. It lists 24 documents for 11 questions on purpose. Only 6 are the subject of a question; the other 18 are distractors, each topically adjacent to a question's source. A corpus where every document is on a different subject makes retrieval look better than it is — any half-working retriever scores a perfect hit rate when there is one candidate per topic. The repo ignores `*.csv`, so the committed test set is un-ignored explicitly: it is source, not scratch data.
87f654b to
7f1f006
Compare
3899a03 to
27e5220
Compare
Part 15 of 15 of the split of #811. Targets
eval/14-ui-page(#825). Docs and data only — no code.What
CLAUDE.md— the Evaluation section: the flow, the layering rule (the service dispatches through a port and never touches Ray), why there are two promptfoo configs, and which values are deliberately not configuration.tests/evaluation/— a ready-made test set: 11 questions over 6 documents, every answer checked against the source rather than generated.Notable
corpus.txtlists the filenames and the README gives the two commands to assemble and upload it.*.csv, so the committed test set is un-ignored explicitly. It is source, not scratch data.Carried over from #811 — known limitations, unchanged
Flagged for a decision rather than fixed:
files_per_minutemeasures serialised throughput, not the system's real concurrency./searchdefaultssimilarity_thresholdto0.75and the eval does not override it, so retrieval can return fewer thantop_kand depress recall for reasons unrelated to ranking.<data_dir>on shared storage — the runner reads dataset files by path.__eval_-prefixed partitions, which are merely hidden from listings.Testing
ruffand the layer-import guard pass. No code changed.