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 embaas embeddings api endpoints #5976

Merged
merged 8 commits into from
Jun 11, 2023

Conversation

juliuslipp
Copy link
Contributor

Introduces embaas embeddings endpoints

In this PR, we add support for embaas embeddings endpoints to Text Embedding Models (with LLMs, and Doc loaders in different PRs coming). We currently offer the MTEB leaderboard top performers, will continue to add top embedding models and soon add support for customers to deploy thier own models.Additional Documentation + Infomation can be found here.

While developing this integration, I closely followed the patterns established by other langchain integrations. Nonetheless, if there are any aspects that require adjustments or if there's a better way to present a new integration, let me know! :)

Who can review?

@eyurtsev eyurtsev added lgtm PR looks good. Use to confirm that a PR is ready for merging. maintainer-to-merge labels Jun 10, 2023
@eyurtsev
Copy link
Collaborator

@juliuslipp thanks for the contribution. This code basically looks perfect.

If you're able to resolve ilnt conflict, we'll be able to merge the code in faster.

Two things that would be nice to have (if you have time/desire):

  1. A unit test that attempts to import the code
  2. Or alternatively a unit test that uses the cod with responses library. It'll make sure that this code will not break if other folks try to enhance it in the future.

langchain/embeddings/embaas.py Show resolved Hide resolved
langchain/embeddings/embaas.py Outdated Show resolved Hide resolved
@juliuslipp
Copy link
Contributor Author

juliuslipp commented Jun 11, 2023

Hey @eyurtsev,

thanks for the quick review. I fixed your comments & the linting.

About the tests, wasn't too sure what you meant by 1., but I freestyled a responses test and hope it works like this :D

Additionally, I added support for a custom api url.

@juliuslipp juliuslipp requested a review from eyurtsev June 11, 2023 10:37
@hwchase17 hwchase17 changed the base branch from master to harrison/embaas June 11, 2023 20:15
@hwchase17 hwchase17 merged commit 7906cfd into langchain-ai:harrison/embaas Jun 11, 2023
rlancemartin pushed a commit that referenced this pull request Jun 13, 2023
# Introduces embaas document extraction api endpoints

In this PR, we add support for embaas document extraction endpoints to
Text Embedding Models (with LLMs, in different PRs coming). We currently
offer the MTEB leaderboard top performers, will continue to add top
embedding models and soon add support for customers to deploy thier own
models. Additional Documentation + Infomation can be found
[here](https://embaas.io).

While developing this integration, I closely followed the patterns
established by other langchain integrations. Nonetheless, if there are
any aspects that require adjustments or if there's a better way to
present a new integration, let me know! :)

Additionally, I fixed some docs in the embeddings integration.

Related PR: #5976 

#### Who can review?
  DataLoaders
  - @eyurtsev
Undertone0809 pushed a commit to Undertone0809/langchain that referenced this pull request Jun 19, 2023
# Introduces embaas document extraction api endpoints

In this PR, we add support for embaas document extraction endpoints to
Text Embedding Models (with LLMs, in different PRs coming). We currently
offer the MTEB leaderboard top performers, will continue to add top
embedding models and soon add support for customers to deploy thier own
models. Additional Documentation + Infomation can be found
[here](https://embaas.io).

While developing this integration, I closely followed the patterns
established by other langchain integrations. Nonetheless, if there are
any aspects that require adjustments or if there's a better way to
present a new integration, let me know! :)

Additionally, I fixed some docs in the embeddings integration.

Related PR: langchain-ai#5976 

#### Who can review?
  DataLoaders
  - @eyurtsev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm PR looks good. Use to confirm that a PR is ready for merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants