-
Notifications
You must be signed in to change notification settings - Fork 25.6k
[DOCS] Fix reranking IA, move retrievers to search api overview #112949
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
Conversation
Documentation preview: |
Another option of course is to remove "Reranking" from "Search your data" [EDIT] this felt like the absolute best approach |
Pinging @elastic/es-docs (Team:Docs) |
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 with a couple of suggestions
@kderusso I pushed an update to provide more context, hopefully it makes sense and is somewhat correct 😄. LMKWYT. Would be nice to have a diagram one day, but probably won't happen in this PR. |
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.
This is looking good so far, thanks for iterating. I have some additional feedback.
[[re-ranking-ranking-overview-hybrid]] | ||
==== Hybrid techniques | ||
|
||
Hybrid search techniques combine results from full-text and vector search pipelines. |
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.
I am not sure how much we have here, but we've been getting more and more questions and feedback on people who want more information on linear boosting as well. I wonder if a sentence here and a link to somewhere else (if we have it) would be worth doing.
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.
In future we want to have fuller docs sections for:
- Full-text search
- ML-powered search
- Hybrid search
I think we'll want to revisit that when the time comes. The goal here is to put LTR and semantic reranking in a smarter IA, first and foremost, not to be the exhaustive source of truth for all these topics.
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.
Sure sure, push back on my scope creep 😀
@@ -0,0 +1,71 @@ | |||
[[re-ranking-overview]] | |||
= Re-ranking |
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.
I'm noticing some inconsistency - sometimes we use Reranking
and other times we use Re-ranking
. I slightly prefer reranking
but am fine with either, though we should do a pass for consistency.
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.
realized that in course of this PR and tried to align on hyphenated version, will need to update the semantic re-reranking page too
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.
should be good after e815f94
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.
FWIW I also prefer reranking
but the hyphenated version seems more au courant :)
Co-authored-by: Kathleen DeRusso <[email protected]>
Co-authored-by: Kathleen DeRusso <[email protected]>
Co-authored-by: Kathleen DeRusso <[email protected]>
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.
Looks great, thanks @leemthompo !
[[re-ranking-ranking-overview-hybrid]] | ||
==== Hybrid techniques | ||
|
||
Hybrid search techniques combine results from full-text and vector search pipelines. |
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.
Sure sure, push back on my scope creep 😀
Co-authored-by: Kathleen DeRusso <[email protected]>
See the URL preview 👁️ 🔭
Summary of this small refactoring:
Screenshots
Before
After (Search your data)
After (Reranking)