Skip to content

Conversation

@Samiul-TheSoccerFan
Copy link
Contributor

As part of Semantic Rerank GA, updating the documentation to address that the inference_id field is not required anymore.

@Samiul-TheSoccerFan Samiul-TheSoccerFan added the >docs General docs changes label Apr 2, 2025
### Prerequisites [_prerequisites_15]

To use `text_similarity_reranker` you must first set up an inference endpoint for the `rerank` task using the [Create {{infer}} API](https://www.elastic.co/docs/api/doc/elasticsearch/operation/operation-inference-put). The endpoint should be set up with a machine learning model that can compute text similarity. Refer to [the Elastic NLP model reference](docs-content://explore-analyze/machine-learning/nlp/ml-nlp-model-ref.md#ml-nlp-model-ref-text-similarity) for a list of third-party text similarity models supported by {{es}}.
To use `text_similarity_reranker` you can use the preconfigured `.rerank-v1-elasticsearch` inference endpoint, which is the default if no `inference_id` is provided. Alternatively, you can set up a custom inference endpoint for the `rerank` task using the [Create {{infer}} API](https://www.elastic.co/docs/api/doc/elasticsearch/operation/operation-inference-put). The endpoint should be set up with a machine learning model that can compute text similarity. Refer to [the Elastic NLP model reference](docs-content://explore-analyze/machine-learning/nlp/ml-nlp-model-ref.md#ml-nlp-model-ref-text-similarity) for a list of third-party text similarity models supported by {{es}}.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should link to our docs on Elastic rerank here

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you mean the Elastic Rerank model page link?

If yes, How about this one?

To use `text_similarity_reranker` you can use the preconfigured `.rerank-v1-elasticsearch` inference endpoint created based on [Elastic Rerank](https://www.elastic.co/guide/en/elasticsearch/reference/current/infer-service-elasticsearch.html#inference-example-elastic-reranker)), which is the default if no `inference_id` is provided.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nvm, you probably meant this documentation link?
https://www.elastic.co/guide/en/machine-learning/current/ml-nlp-rerank.html

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like your link to our model, let's find a way to add both links?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated the documentation to include both link. Please let me know what do you think

@Samiul-TheSoccerFan Samiul-TheSoccerFan marked this pull request as ready for review April 7, 2025 19:45
@elasticsearchmachine elasticsearchmachine added the Team:Docs Meta label for docs team label Apr 7, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@Samiul-TheSoccerFan Samiul-TheSoccerFan requested a review from a team April 7, 2025 19:46
Copy link
Contributor

@Mikep86 Mikep86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We'll need to backport this change to 9.0, right? And we'll need a separate PR for 8.18/8.19 due to the docs format shift.

Copy link
Member

@kderusso kderusso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Samiul-TheSoccerFan
Copy link
Contributor Author

@elasticmachine update branch

@Samiul-TheSoccerFan Samiul-TheSoccerFan added the auto-backport Automatically create backport pull requests when merged label Apr 16, 2025
@Samiul-TheSoccerFan Samiul-TheSoccerFan merged commit 2e1101c into elastic:main Apr 16, 2025
6 checks passed
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
9.0

Samiul-TheSoccerFan added a commit to Samiul-TheSoccerFan/elasticsearch that referenced this pull request Apr 16, 2025
* Updating text_similarity_reranker documentation

* Updating docs to include urls

* remove extra THE from the text

---------

Co-authored-by: Elastic Machine <[email protected]>
elasticsearchmachine pushed a commit that referenced this pull request Apr 16, 2025
* Updating text_similarity_reranker documentation

* Updating docs to include urls

* remove extra THE from the text

---------

Co-authored-by: Elastic Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged >docs General docs changes Team:Docs Meta label for docs team v9.0.0 v9.0.1 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants