-
Notifications
You must be signed in to change notification settings - Fork 14
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
Changes related to Meilisearch v1.6.0 #294
Comments
I verified that at a minimum the python tests will need updating for vector search.
meilisearch.errors.MeilisearchApiError: MeilisearchApiError. Error code: missing_search_hybrid. Error message: Invalid request: missing `hybrid` parameter when both `q` and `vector` are present.. Hint: It might not be working because you're not up to date with the Meilisearch version that search call requires. Error documentation: https://docs.meilisearch.com/errors#missing_search_hybrid Error type: invalid_request |
900: Changes related to the next Meilisearch release (v1.6.0) r=curquiza a=meili-bot Related to this issue: meilisearch/integration-guides#294 This PR: - gathers the changes related to the next Meilisearch release (v1.6.0) so that this package is ready when the official release is out. - should pass the tests against the [latest pre-release of Meilisearch](https://github.com/meilisearch/meilisearch/releases). - might eventually contain test failures until the Meilisearch v1.6.0 is out.⚠️ This PR should NOT be merged until the next release of Meilisearch (v1.6.0) is out. _This PR is auto-generated for the [pre-release week](https://github.com/meilisearch/integration-guides/blob/main/resources/pre-release-week.md) purpose._ Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com> Co-authored-by: curquiza <clementine@meilisearch.com> Co-authored-by: Clémentine U. - curqui <clementine@meilisearch.com> Co-authored-by: Paul Sanders <psanders1@gmail.com>
503: Update tests related to the next Meilisearch release (v1.6.0) r=curquiza a=meili-bot Related to this issue: meilisearch/integration-guides#294 This PR: - gathers the changes related to the next Meilisearch release (v1.6.0) so that this package is ready when the official release is out. - should pass the tests against the [latest pre-release of Meilisearch](https://github.com/meilisearch/meilisearch/releases). - might eventually contain test failures until the Meilisearch v1.6.0 is out.⚠️ This PR should NOT be merged until the next release of Meilisearch (v1.6.0) is out. _This PR is auto-generated for the [pre-release week](https://github.com/meilisearch/integration-guides/blob/main/resources/pre-release-week.md) purpose._ Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com> Co-authored-by: curquiza <clementine@meilisearch.com> Co-authored-by: Clémentine U. - curqui <clementine@meilisearch.com>
368: Update tests related to the next Meilisearch release (v1.6.0) r=curquiza a=meili-bot Related to this issue: meilisearch/integration-guides#294 This PR: - gathers the changes related to the next Meilisearch release (v1.6.0) so that this package is ready when the official release is out. - should pass the tests against the [latest pre-release of Meilisearch](https://github.com/meilisearch/meilisearch/releases). - might eventually contain test failures until the Meilisearch v1.6.0 is out.⚠️ This PR should NOT be merged until the next release of Meilisearch (v1.6.0) is out. _This PR is auto-generated for the [pre-release week](https://github.com/meilisearch/integration-guides/blob/main/resources/pre-release-week.md) purpose._ Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com> Co-authored-by: curquiza <clementine@meilisearch.com> Co-authored-by: Clémentine U. - curqui <clementine@meilisearch.com>
600: Changes related to the next Meilisearch release (v1.6.0) r=brunoocasali a=meili-bot Related to this issue: meilisearch/integration-guides#294 This PR: - gathers the changes related to the next Meilisearch release (v1.6.0) so that this package is ready when the official release is out. - should pass the tests against the [latest pre-release of Meilisearch](https://github.com/meilisearch/meilisearch/releases). - might eventually contain test failures until the Meilisearch v1.6.0 is out.⚠️ This PR should NOT be merged until the next release of Meilisearch (v1.6.0) is out. _This PR is auto-generated for the [pre-release week](https://github.com/meilisearch/integration-guides/blob/main/resources/pre-release-week.md) purpose._ Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com> Co-authored-by: Bruno Casali <brunoocasali@gmail.com> Co-authored-by: Clémentine U. - curqui <clementine@meilisearch.com>
900: Update tests related to the next Meilisearch release (v1.6.0) r=curquiza a=meili-bot Related to this issue: meilisearch/integration-guides#294 This PR: - gathers the changes related to the next Meilisearch release (v1.6.0) so that this package is ready when the official release is out. - should pass the tests against the [latest pre-release of Meilisearch](https://github.com/meilisearch/meilisearch/releases). - might eventually contain test failures until the Meilisearch v1.6.0 is out.⚠️ This PR should NOT be merged until the next release of Meilisearch (v1.6.0) is out. _This PR is auto-generated for the [pre-release week](https://github.com/meilisearch/integration-guides/blob/main/resources/pre-release-week.md) purpose._ Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com> Co-authored-by: curquiza <clementine@meilisearch.com> Co-authored-by: Clémentine U. - curqui <clementine@meilisearch.com> Co-authored-by: Paul Sanders <psanders1@gmail.com>
900: Update tests related to the next Meilisearch release (v1.6.0) r=curquiza a=meili-bot Related to this issue: meilisearch/integration-guides#294 This PR: - gathers the changes related to the next Meilisearch release (v1.6.0) so that this package is ready when the official release is out. - should pass the tests against the [latest pre-release of Meilisearch](https://github.com/meilisearch/meilisearch/releases). - might eventually contain test failures until the Meilisearch v1.6.0 is out.⚠️ This PR should NOT be merged until the next release of Meilisearch (v1.6.0) is out. _This PR is auto-generated for the [pre-release week](https://github.com/meilisearch/integration-guides/blob/main/resources/pre-release-week.md) purpose._ Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com> Co-authored-by: curquiza <clementine@meilisearch.com> Co-authored-by: Clémentine U. - curqui <clementine@meilisearch.com> Co-authored-by: Paul Sanders <psanders1@gmail.com>
219: Changes related to the next Meilisearch release (v1.6.0) r=brunoocasali a=meili-bot Related to this issue: meilisearch/integration-guides#294⚠️ This PR should NOT be merged until the next release of Meilisearch (v1.6.0) is out. _This PR is auto-generated for the [pre-release week](https://github.com/meilisearch/integration-guides/blob/main/resources/pre-release-week.md) purpose._ Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com> Co-authored-by: Bruno Casali <brunoocasali@gmail.com> Co-authored-by: brunoocasali <brunoocasali@users.noreply.github.com>
1614: Changes related to the next Meilisearch release (v1.6.0) r=curquiza a=meili-bot Related to this issue: meilisearch/integration-guides#294 This PR: - gathers the changes related to the next Meilisearch release (v1.6.0) so that this package is ready when the official release is out. - should pass the tests against the [latest pre-release of Meilisearch](https://github.com/meilisearch/meilisearch/releases). - might eventually contain test failures until the Meilisearch v1.6.0 is out.⚠️ This PR should NOT be merged until the next release of Meilisearch (v1.6.0) is out. _This PR is auto-generated for the [pre-release week](https://github.com/meilisearch/integration-guides/blob/main/resources/pre-release-week.md) purpose._ Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com> Co-authored-by: Morgane Dubus <30866152+mdubus@users.noreply.github.com> Co-authored-by: Clémentine U. - curqui <clementine@meilisearch.com> Co-authored-by: Morgane Dubus <morgane.d@meilisearch.com>
Issues opened for Are k8s and cloud providers released @brunoocasali? Also missing this PR to be merged on the dart side. Having an issue with linter 😢 |
368: Update tests related to the next Meilisearch release (v1.6.0) r=curquiza a=meili-bot Related to this issue: meilisearch/integration-guides#294 This PR: - gathers the changes related to the next Meilisearch release (v1.6.0) so that this package is ready when the official release is out. - should pass the tests against the [latest pre-release of Meilisearch](https://github.com/meilisearch/meilisearch/releases). - might eventually contain test failures until the Meilisearch v1.6.0 is out.⚠️ This PR should NOT be merged until the next release of Meilisearch (v1.6.0) is out. _This PR is auto-generated for the [pre-release week](https://github.com/meilisearch/integration-guides/blob/main/resources/pre-release-week.md) purpose._ Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com> Co-authored-by: curquiza <clementine@meilisearch.com> Co-authored-by: Clémentine U. - curqui <clementine@meilisearch.com> Co-authored-by: Ahmed Fwela <ahmednfwela@bdaya-dev.com>
Discussed with @brunoocasali, k8s and cloud providers are done Closing this issue then |
is this error fixed now? I'm still getting this erro? |
Which SDK are you using? Which version of Meiliesarch? What is the error message? |
I've got the same error when I do vector search using python sdk 0.30.0 and the version of Meilisearch is 1.6.2: |
Did you include the |
@sanders41 I'm following this tutorial and I'm get the same error and there is nothing about the error code in the docs. https://blog.meilisearch.com/langchain-semantic-search-tutorial/ |
There were breaking changes to vector search in 1.6. I have not used LangChain and don't know much about it, but looking at the example provided it doesn't look like LangChain is compatible with Meilisearch 1.6+ vector search. @Strift I'm not sure if LangChain needs to be updated, if the tutorial needs updating to mention it works with Meilisearch >=1.3 <1.6, or if it is possible somehow to make it work with Meilisearch 1.6+? |
I tried to add an embedder to my meilisearch instance but I got the error bellow. When we add an embedded to meili, it will generate vectors for all documents or only use to embedding queries?
|
Hello @kauly, If you want to use LangChain, please use a previous Meilisearch version, like v1.5. Our current LangChain integration is intended for v1.3, v1.4, and v1.5. @CaroFG is currently updating our LangChain integration. After this, we'll update the tutorial too. 🙏 Thanks @sanders41 for the heads up. |
This issue gathers the changes related to the v1.6.0 of Meilisearch that will impact the integrations scope.
📅 Release date: 15/01/2024
TODO
Before pre-release
#1
integrations.Integration to update:
Pre-release
rc0
, define in which SDKs we have to update the testsRelease day
Features to implement
proximityPrecision
settingsIssue: meilisearch/meilisearch#4187
Usage: https://www.notion.so/meilisearch/Customize-proximity-precision-usage-aa69c2bab2c3402bab9340ae4def4577
Should be implemented for
Vector search v2
Usage: https://www.notion.so/meilisearch/v1-6-Hybrid-Search-Embedders-ea42c82f90cc4bc0be1eeb917c1118c8
By Meili developers
By community if available
->⚠️ If not implemented by community, we will have to be clear (release notes + issues) the experimental vector search feature is broken in the SDK until someone implements it.
The text was updated successfully, but these errors were encountered: