-
-
Notifications
You must be signed in to change notification settings - Fork 117
[Store] Add SurrealDB #139
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
Conversation
8c9ad55 to
e9b7322
Compare
|
We just created some Thanks |
chr-hertel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're really on a great streak here to level up the store bridges - thanks for that! 🙏
Somehow the example doesn't work for me again - see below
e9b7322 to
e38a096
Compare
|
@OskarStark 👋🏻 I updated the |
|
Needs a rebase please |
chr-hertel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
small things about the inner contract of the Store implementation here
e38a096 to
23ab86a
Compare
23ab86a to
6e82e2f
Compare
6e82e2f to
a14c19d
Compare
|
Thank you @Guikingone. |
Hi 👋🏻
This PR aim to introduce
SurrealDBas a store for embeddings, the store use theHTTPAPI as the PHP SDK is ugly to use (and does not allow mocks), the tests needs to be improved / completed but the main implementation is ready.PS: By default, we can't send more than
1275embeddings during the insertion phase, the SDK can bypass this limitation but again, testing issues, API and more so, the road to follow is up to the team.