meilisearch / meilisearch-js

JavaScript client for the Meilisearch API

https://www.meilisearch.com

MIT License

703 stars 85 forks source link

feat: hybrid search improvements for v1.8.x #1647

Closed mdubus closed 2 months ago

mdubus commented 2 months ago

Pull Request

Related issue

Fixes #1641

What does this PR do?

Removal of the vector and _semanticScore fields from the SearchResponse
Addition of a new distribution field to all embedders
Addition of 2 new embedders: REST and Ollama
Update failing tests & snapshots
Addition of tests + missing tests

⚠️ CI failing

CI fails because of 2 things:

The distribution field is not retrieved correctly on the settings/embedders routes (see https://github.com/meilisearch/meilisearch/issues/4594)
The Ollama embedder isn't retrieved completely (see https://github.com/meilisearch/meilisearch/issues/4595 )

Once those 2 things are fixed, it should be fine 🙌

PR checklist

Please check if your PR fulfills the following requirements:

[x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[x] Have you read the contributing guidelines?
[x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!