meilisearch-java icon indicating copy to clipboard operation
meilisearch-java copied to clipboard

[v1.3] Display hits ranking scores

Open brunoocasali opened this issue 1 year ago • 1 comments

⚠️ This issue is generated, it means the examples and the namings do not necessarily correspond to the language of this repository. Also, if you are a maintainer, please add any clarification and instructions about this issue.

Sorry if this is already wholly/partially implemented. Feel free to let me know about the state of this issue in the repo.

Related to https://github.com/meilisearch/integration-guides/issues/280


This issue is divided into two sections, first, you need to make the implementation, and second, you must update the code-samples (no one likes outdated docs, right?).

New implementation

Related to:

  • issue: https://github.com/meilisearch/meilisearch/issues/3771
  • discussion: https://github.com/meilisearch/product/discussions/594
  • spec: https://github.com/meilisearch/specifications/pull/252

This feature aims to return ranking details for each document to understand and tweak the score of the documents more efficiently.

Ensure the SDKs can handle the new search parameter showRankingScore. Also, ensure the SDK can handle the _rankingScore attribute in the matched hits.

Code samples

Inside of this file: .code-samples.meilisearch.yml:

Create a new entry with this key search_parameter_guide_show_ranking_score_1 Refer to this curl example and translate it by using the java methods: https://github.com/meilisearch/documentation/blob/e4fdc3d485c22e31a045b8a71bacf18b6a7d50e0/.code-samples.meilisearch.yaml#L1095

TODO:

  • [ ] Add the ability receive a new param in the search method called showRankingScore.
  • [ ] Add the ability handle the new _rankingScore key/value in the search hits' response.
  • [ ] Update the code-samples accordingly
  • [ ] Add integration tests

brunoocasali avatar Aug 08 '23 17:08 brunoocasali

Looks like it has been added to https://github.com/meilisearch/meilisearch-java/pull/652/files but the issue is not fully fixed, so contributions are still welcome to completely solve it 😊

curquiza avatar Feb 13 '24 11:02 curquiza