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

feat: add /similarity route #331

Merged
merged 1 commit into from
Jul 8, 2024
Merged

Conversation

OlivierDehaene
Copy link
Member

Required for compatibility.

@OlivierDehaene OlivierDehaene merged commit e496fe7 into main Jul 8, 2024
2 of 4 checks passed
@OlivierDehaene OlivierDehaene deleted the feat/sentence_similarity branch July 8, 2024 10:24
@cordawyn
Copy link

cordawyn commented Jul 9, 2024

@OlivierDehaene Would it make sense to make it compatible with huggingface-hub client: https://github.com/huggingface/huggingface_hub/blob/ca111fb51193968faed548a4b865c79736822aed/src/huggingface_hub/inference/_client.py#L1416 ? It looks like they expect a bit different names for the attributes of SimilarityInput (

pub(crate) struct SimilarityInput {
).

@OlivierDehaene
Copy link
Member Author

What do you mean? It is compatible.

@OlivierDehaene
Copy link
Member Author

@cordawyn
Copy link

cordawyn commented Jul 9, 2024

Oh, my apologies, I must've looked at a different revision or something (probably got confused with the python method args). Thanks for the quick reply!

pi314ever pushed a commit to pi314ever/tei-gaudi that referenced this pull request Aug 21, 2024
pi314ever pushed a commit to pi314ever/tei-gaudi that referenced this pull request Aug 26, 2024
MasakiMu319 pushed a commit to MasakiMu319/text-embeddings-inference that referenced this pull request Nov 27, 2024
aagnone3 pushed a commit to StratisLLC/hf-text-embeddings-inference that referenced this pull request Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants