Skip to content
This repository was archived by the owner on Jul 16, 2025. It is now read-only.

chore: Allow arbitrary vector dimensions for MariaDB store #348

Merged
merged 1 commit into from
Jun 27, 2025

Conversation

valtzu
Copy link
Contributor

@valtzu valtzu commented Jun 26, 2025

MariaDB insists the dimensions of the vector being inserted match exactly the dimensions of the vector defined in the schema.

Since the store knows nothing about the Model & it's expected vector dimensions, let's allow passing vector dimensions as a parameter to initialize() leave the problem to the user.

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.

LGTM, thank you @valtzu 👍

@chr-hertel chr-hertel merged commit a42da3a into php-llm:main Jun 27, 2025
7 checks passed
OskarStark added a commit to symfony/ai that referenced this pull request Jul 1, 2025
…(valtzu)

This PR was merged into the main branch.

Discussion
----------

chore: Allow arbitrary vector dimensions for MariaDB store

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

Cherry picking php-llm/llm-chain#348

> MariaDB insists the dimensions of the vector being inserted match exactly the dimensions of the vector defined in the schema.

Since the store knows nothing about the Model & it's expected vector dimensions, let's allow passing vector dimensions as a parameter to `initialize()` leave the problem to the user.

Commits
-------

b1c2fe5 chore: Allow arbitrary vector dimensions for MariaDB store (#348)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants