Skip to content

[Store] Change vectorizeDocuments method to __invoke in Vectorizer #470

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 5, 2025
Merged

[Store] Change vectorizeDocuments method to __invoke in Vectorizer #470

merged 1 commit into from
Sep 5, 2025
+13 −13

Conversation

OskarStark
Copy link
Contributor

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

This makes the Vectorizer class invokable, allowing it to be called directly as a function while maintaining the same functionality.

@OskarStark OskarStark self-assigned this Sep 5, 2025
@OskarStark OskarStark added Store Issues & PRs about the AI Store component BC Break Breaking the Backwards Compatibility Promise labels Sep 5, 2025
@OskarStark OskarStark force-pushed the vectorizer-invoke-method branch from cfa96bd to c41dae0 Compare September 5, 2025 15:04
@OskarStark OskarStark merged commit 4fae664 into symfony:main Sep 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chr-hertel chr-hertel Awaiting requested review from chr-hertel chr-hertel is a code owner

@Nyholm Nyholm Awaiting requested review from Nyholm Nyholm is a code owner

Assignees

@OskarStark OskarStark

Labels
BC Break Breaking the Backwards Compatibility Promise Feature New feature Status: Needs Review Store Issues & PRs about the AI Store component
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@OskarStark @carsonbot