Skip to content

Conversation

@Guikingone
Copy link
Contributor

@Guikingone Guikingone commented Jul 16, 2025

Q A
Bug fix? no
New feature? yes
Docs? yes
Issues None
License MIT

Hi 👋🏻

This PR aim to introduce SurrealDB as a store for embeddings, the store use the HTTP API 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 1275 embeddings 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.

@chr-hertel chr-hertel added the Store Issues & PRs about the AI Store component label Jul 16, 2025
@OskarStark
Copy link
Contributor

We just created some CHANGELOG.md files.
Please rebase your branch and add info to the appropriate CHANGELOG.md file.

Thanks

Copy link
Member

@chr-hertel chr-hertel left a 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

@Guikingone
Copy link
Contributor Author

@OskarStark 👋🏻

I updated the CHANGELOG.md file for store 🙂

@OskarStark
Copy link
Contributor

Needs a rebase please

Copy link
Member

@chr-hertel chr-hertel left a 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

@chr-hertel
Copy link
Member

Thank you @Guikingone.

@chr-hertel chr-hertel merged commit 0f569f4 into symfony:main Jul 20, 2025
13 checks passed
@Guikingone Guikingone deleted the store/surrealdb branch August 1, 2025 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Store Issues & PRs about the AI Store component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants