-
Notifications
You must be signed in to change notification settings - Fork 102
[ML]rename rerank score to relevance_score #3751
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
Hi! @davidkyle , could u please take a look at 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.
LGTM
Thanks for the fix @Huaixinww. It should have be relevance_score
all along, the API never return a score
field
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.
Correctly validated in #3755, LGTM
(cherry picked from commit f630538)
(cherry picked from commit f630538)
(cherry picked from commit f630538)
(cherry picked from commit f630538)
thanks @Huaixinww for the investigation and the fix! |
(cherry picked from commit f630538) Co-authored-by: Huaixinww <[email protected]>
(cherry picked from commit f630538) Co-authored-by: Huaixinww <[email protected]>
(cherry picked from commit f630538) Co-authored-by: Huaixinww <[email protected]>
(cherry picked from commit f630538) Co-authored-by: Huaixinww <[email protected]>
When we call the inference rerank interface using the Elasticsearch Java client, we encountered the following error:
We found that the relevant serialization code in the Elasticsearch Java client is as follows:
The RankedDocument in the Elasticsearch Java client requires a non-null score, but the rerank interface returns a relevance_score (which is also documented here: Perform inference API Rerank Example).
The Elasticsearch Java client indicates that this code is generated according to the Elasticsearch API specification:
Therefore, should we change the
score
value here torelevance_score
?Steps to Reproduce
Elasticsearch version: 8.15.1
dependencies:
We referred to this document: Getting Started with Elasticsearch Java Client