Skip to content

fix(eval): rank a tied document by id before insertion order - #1606

Open
SashaMIT wants to merge 1 commit into
FlagOpen:masterfrom
SashaMIT:fix/mrr-tie-break
Open

SashaMIT wants to merge 1 commit into
FlagOpen:masterfrom
SashaMIT:fix/mrr-tie-break

Conversation

@SashaMIT

Copy link
Copy Markdown

MRR ranked tied documents by the order they appeared in the dict. A relevant document b that tied with an earlier a scored 0 at cutoff 1. Ties now break by document id, higher first, in both MRR and capped recall. A higher score still ranks first.

test_a_tied_relevant_doc_is_ranked_by_id

Before the change, MRR@1 was 0.0. After it, MRR@1 and R_cap@1 are 1.0.

MRR put the earlier dict entry first when scores were equal, so a relevant document that tied scored 0 at cutoff 1.

This branch has not been deployed

No deployments
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