-
Notifications
You must be signed in to change notification settings - Fork 463
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
[FEATURE REQUEST]: Being able to handle embeddings with Bedrock #2539
Comments
Hi @boroju happy to help! There are some examples in the |
Hey @blythed, perfect! I'll look into it and reaching out in case of further questions/doubts. Thanks for replying! |
Hey Timo, hope you well Where can I find the classes "Table" or "Collection" within the superduper framework? According to your documentation, they should be here: from superduper import Table https://docs.superduper.io/docs/execute_api/data_encodings_and_schemas#create-a-table-with-a-schema Previously, when the framework was "superduperdb," we were able to import "Collection" this way: BUT I couldn't find either "Collection" or "Table" as described in your current documentation. I think you should take a look. 👀 This is basically what I'm trying to do:
And I am getting this error:
The package version I am using is: I know this is the latest available version because I just added it.The same functionality was working before with the older "superduperdb" framework version. Please see our previous code below:
I'm going to add this comment as part of the open issue: I would like to point out that in the /templates directory, there is nothing for connecting to Bedrock and generating embeddings from one of the models there. This was mentioned to me by @blythed in response. If you want to connect and pair up to fix/debug the code, I’m willing to do so with you. 😊 Thank you! |
Hi @boroju we are about to release 0.4.0 in the next couple of days. In this new version, adding pdfs (and other data) is much easier than before, and is decoupled from which data-backend you would use. Here is a pdf example, which you can play with directly/ update in the notebook: https://github.com/superduper-io/superduper/tree/main/templates/pdf_rag/build.ipynb. This is similar code to the code we used to build the "volvo" demo. I hope that helps! |
Hi @blythed, I've been following mentioned examples, and now I am looking at the "volvo" demo, which uses the old framework version. I am wondering what the alternative is now for the unstructured encoder:
The line above is no longer supported in the newer version of superduper. The same issue occurs with "Collection," but I believe I can use "Table" instead:
Would you mind clarifying this for me? |
@blythed This is basically what I'm trying to do here:
|
Contact Details
[email protected]; [email protected]
Feature Description
Leveraging Bedrock for Embeddings in "Insurance PDF Search"
Use Case Description
I'm a member of the Industry Solutions team at MongoDB. Today, we had a meeting with Timo where we discussed the possibility of refactoring our existing 'Insurance PDF Search' demo to use Bedrock embeddings instead of the OpenAI ones (OpenAIEmbedding). The code, which was written several months ago, still calls the 'superduperdb' library, which I believe is legacy since the library/company name has changed.
Considering our desire to move away from OpenAI embeddings and start leveraging Bedrock, perhaps we can collaborate to make this happen. By the way, Timo mentioned that we should open a feature request to explore how to handle this using 'plugins'.
Organization
Refactoring our existing 'Insurance PDF Search' demo, to use Bedrock embeddings and remove legacy imports/dependencies calling to superduperdb.
Who are the stake-holders?
MongoDB Industry Solutions team, Luca Napoli [email protected], Jeff Needham [email protected]
The text was updated successfully, but these errors were encountered: