-
Notifications
You must be signed in to change notification settings - Fork 478
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Update MS MARCO V2 regressions #1967
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1967 +/- ##
=========================================
Coverage 58.30% 58.30%
Complexity 1067 1067
=========================================
Files 186 186
Lines 10147 10147
Branches 1398 1398
=========================================
Hits 5916 5916
Misses 3758 3758
Partials 473 473 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
src/main/resources/regression/msmarco-v2-passage-unicoil-0shot.yaml
Outdated
Show resolved
Hide resolved
@yuki617 @MXueguang @ToluClassics Ready for review! |
seems some regression documents didn't provide instructions to download corpus? |
Yea... that' unrelated to this specific PR, so let me circle back and do that later? |
R@1000: | ||
- 0.7731 | ||
- 0.7698 | ||
- name: rocchio |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe unicoil-0shot+rocchio is a better name?
Add RM3 and Rocchio in cases where it's missing.