Skip to content
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

Fix tests by passing embedder #1020

Merged
merged 2 commits into from
Sep 30, 2024
Merged

Conversation

dureuill
Copy link

Pull Request

Related issue

Fixes #1019

What does this PR do?

  • Change comment stating the default model for OpenAI
  • Add document_template_max_bytes to models
  • Fix tests

Copy link
Collaborator

@sanders41 sanders41 left a comment

Choose a reason for hiding this comment

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

lgtm

bors merge

meili-bors bot added a commit that referenced this pull request Sep 30, 2024
1020: Fix tests by passing embedders r=sanders41 a=dureuill

# Pull Request

## Related issue
Fixes #1019 

## What does this PR do?
- Change comment stating the default model for OpenAI
- Add `document_template_max_bytes` to models
- Fix tests



Co-authored-by: Louis Dureuil <[email protected]>
Copy link
Contributor

meili-bors bot commented Sep 30, 2024

Build failed:

@dureuill dureuill changed the title Fix tests by passing embedders Fix tests by passing embedder Sep 30, 2024
@dureuill
Copy link
Author

dureuill commented Sep 30, 2024

ah, the tests don't pass against the latest meili because it doesn't have (yet) documentTemplateMaxBytes, which is a v1.11 novelty.

well at least we're ready 😄

@sanders41
Copy link
Collaborator

Yes. @curquiza I assume it is OK for me to manually merge this instead of using bors? Bors is trying to test against latest release.

@curquiza
Copy link
Member

yes manually merge is needed here 😉

@curquiza curquiza merged commit 2d1c0b5 into bump-meilisearch-v1.11.0 Sep 30, 2024
10 of 11 checks passed
@curquiza curquiza deleted the embedder-is-mandatory branch September 30, 2024 13:17
@curquiza curquiza linked an issue Sep 30, 2024 that may be closed by this pull request
2 tasks
@sanders41 sanders41 added the enhancement New feature or request label Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[v1.11.0] Fix AI-powered search feature following breaking changes
3 participants