Update sparse text embeddings API route for Inference Service#118025
Update sparse text embeddings API route for Inference Service#118025demjened merged 4 commits intoelastic:mainfrom
Conversation
|
Hi @demjened, I've created a changelog YAML for you. |
|
Pinging @elastic/search-inference-team (Team:Search - Inference) |
|
Pinging @elastic/search-eng (Team:SearchOrg) |
maxjakob
left a comment
There was a problem hiding this comment.
Hold on a minute, are we sure we want the task type to be different to the Inference API? It is called sparse_embedding there.
|
@maxjakob IMO there is no need for the downstream Inference Service path to match the path of the Inference API - that's an implementation detail in the model class. Also, we're adding So in EIS we can adopt either naming pattern as long as we're consistent. |
💔 Backport failed
You can use sqren/backport to manually backport by running |
Per discussion with the Search Inference team, we're modifying the API path of ELSER embeddings in Elastic Inference Service:
/api/v1/sparse-text-embedding/ELSERv2->/api/v1/sparse-text-embedding*s*/ELSERv2.