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

Add HuggingFace Hub embeddings support #86

Closed
hwchase17 opened this issue Nov 8, 2022 · 2 comments · Fixed by #125
Closed

Add HuggingFace Hub embeddings support #86

hwchase17 opened this issue Nov 8, 2022 · 2 comments · Fixed by #125
Labels
03 enhancement Enhancement of existing functionality good first issue Good for newcomers

Comments

@hwchase17
Copy link
Contributor

No description provided.

@hwchase17 hwchase17 added 03 enhancement Enhancement of existing functionality good first issue Good for newcomers labels Nov 8, 2022
@baskaryan
Copy link
Collaborator

ooc do we prefer using python libraries to direct http access of apis? latter would require fewer dependencies but guess you may have to do more maintenance if apis change

@hwchase17
Copy link
Contributor Author

good question. if there exists a good sdk im partial to using that

baskaryan added a commit that referenced this issue Nov 27, 2022
Add support for calling HuggingFace embedding models
using the HuggingFaceHub Inference API. New class mirrors
the existing HuggingFaceHub LLM implementation. Currently
only supports 'sentence-transformers' models.

Closes #86
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
03 enhancement Enhancement of existing functionality good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants