Skip to content

fix: count retrieval misses in engine rank denominators - #1

Open
rudycelekli wants to merge 1 commit into
firecrawl:mainfrom
rudycelekli:fix/engine-ranking-full-denominator
Open

rudycelekli wants to merge 1 commit into
firecrawl:mainfrom
rudycelekli:fix/engine-ranking-full-denominator

Conversation

@rudycelekli

Copy link
Copy Markdown

score_pass() divides repo/fix engine rank-1 and MRR totals by hits rather than all attempted items. Ranks [1, 2, miss, timeout] therefore report MRR 0.75 rather than 0.375. Use the full denominator consistently with the documented rule and docs track; preserve undefined values for native arms without ranked observations.

Nine synthetic regressions cover arithmetic, misses, timeout, depth cap, all-miss and native cases. Against exact upstream production: 6 fail, 3 pass. Fixed full Python 3.11 suite: 141 passed. Base-to-head diff check passes. No paid API calls or published result tables changed; historical engine diagnostic metrics need rescoring.

AI-assisted contribution; code and regression evidence reviewed before submission.

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