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

feat: add pinecone store #55

Merged
merged 1 commit into from
Sep 26, 2024
Merged

feat: add pinecone store #55

merged 1 commit into from
Sep 26, 2024

Conversation

chr-hertel
Copy link
Member

No description provided.

@chr-hertel chr-hertel merged commit ec118a0 into main Sep 26, 2024
5 checks passed
@chr-hertel chr-hertel deleted the pinecone-support branch September 26, 2024 21:48
@@ -126,7 +126,7 @@ public function query(Vector $vector, array $options = []): array

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.

None yet

2 participants