-
Notifications
You must be signed in to change notification settings - Fork 468
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Install RAG24 baseline + update docs #2638
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2638 +/- ##
=========================================
Coverage 65.59% 65.59%
Complexity 1480 1480
=========================================
Files 225 225
Lines 13010 13010
Branches 1551 1551
=========================================
Hits 8534 8534
Misses 3949 3949
Partials 527 527 ☔ View full report in Codecov by Sentry. |
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.
One main point, I think besides this its good to go.
README.md
Outdated
@@ -291,12 +291,28 @@ Key: | |||
|
|||
</details> | |||
<details> | |||
<summary>MS MARCO V2.1 Document Regressions</summary> | |||
<summary>MS MARCO V2.1 Passage Regressions (TREC RAG 24)</summary> |
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.
Terminology here can be misleading, if we are doing segmented maybe we should say that instead of passage?
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.
lgtm
Note that rag2024 regression doesn't actually work yet, we're blocked on #2639
However, since @UShivani3 is currently away, I'd like to get his merged first and then circle back to fix.