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.

Sign up for GitHub

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

Jump to bottom

Add Hugging Face Inference Endpoints as a rest embedder guide #3053

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add Hugging Face Inference Endpoints as a rest embedder guide #3053

wants to merge 4 commits into from

Conversation

gmourier
Copy link
Member

@gmourier gmourier commented Nov 15, 2024 •
edited
Loading

What does this PR do?

  • Add a guide on how to use a model deployed on Hugging Face using Meilisearch rest embedder setting
  • Fix a minor mistake in the Cloudflare one

@meili-bot
Copy link
Collaborator

How to see the preview of this PR?

⚠️ Private link, only accessible to Meilisearch employees.

Go to this URL: https://website-git-deploy-preview-mei-16-meili.vercel.app/docs/branch:add-hugging-face-inference-endpoint-guide

Credentials to access the page are in the company's password manager as "Docs deploy preview".

qdequele
qdequele previously approved these changes Nov 18, 2024
View reviewed changes
guimachiavelli
guimachiavelli requested changes Nov 19, 2024
View reviewed changes
guides/embedders/huggingface.mdx Outdated Show resolved Hide resolved
guides/embedders/huggingface.mdx Outdated Show resolved Hide resolved
guides/embedders/huggingface.mdx Outdated Show resolved Hide resolved
guides/embedders/huggingface.mdx Outdated Show resolved Hide resolved
guides/embedders/huggingface.mdx Outdated Show resolved Hide resolved
guides/embedders/huggingface.mdx
@@ -0,0 +1,87 @@
---
title: Semantic Search with Hugging Face Inference Endpoints - Meilisearch documentation
Copy link
Member

Choose a reason for hiding this comment

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

Why are we focussing on semantic search specifically for this one, instead of going with hybrid?

Copy link
Member Author

Choose a reason for hiding this comment

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

I've reused the snippet used by similar guides like Cohere, Cloudflare, etc... to keep consistency, we can definitely go with the hybrid instead.

guides/embedders/huggingface.mdx Outdated Show resolved Hide resolved
guides/embedders/huggingface.mdx Outdated Show resolved Hide resolved
guides/embedders/huggingface.mdx Outdated Show resolved Hide resolved
guides/embedders/huggingface.mdx Outdated Show resolved Hide resolved
@gmourier
Copy link
Member Author

Thanks for the review, @guimachiavelli; we definitely need to explain somewhere how to leverage the rest embedder to adapt to a wide variety of models API!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@qdequele qdequele qdequele left review comments

@guimachiavelli guimachiavelli Awaiting requested review from guimachiavelli

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@gmourier @meili-bot @guimachiavelli @qdequele