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

Conversation

@chr-hertel
Copy link
Member

No description provided.

@chr-hertel chr-hertel merged commit ec118a0 into main Sep 26, 2024
@chr-hertel chr-hertel deleted the pinecone-support branch September 26, 2024 21:48

foreach ($results as $result) {
$documents[] = Document::fromVector(
Vector::create1536($result[$this->vectorFieldName]),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So the static one is still needed?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope, just thought it is an unnecessary Detail and limitation here

@OskarStark
Copy link
Contributor

Nice one

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants