[PR #107] [CLOSED] fix bug: collection name initialize in PGVector class #169

Closed
opened 2026-02-16 05:16:50 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langchain-postgres/pull/107
Author: @freemjstudio
Created: 8/22/2024
Status: Closed

Base: mainHead: collection_name_param


📝 Commits (1)

  • cfebc2a fix bug: collection name initialize in PGVector class

📊 Changes

1 file changed (+5 additions, -2 deletions)

View changed files

📝 langchain_postgres/vectorstores.py (+5 -2)

📄 Description


vectorstore = PGVector(
        embeddings=embeddings,
        collection_name=collection_name,
        connection=connection,
        use_jsonb=True
    )

I found a small bug when I created PGVector class. I wanted to use Table that I already created but in original code, PGVector initializes collection_name with default collection name variable :_LANGCHAIN_DEFAULT_COLLECTION_NAME.

I hope you check this out ! Thank you .


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/langchain-postgres/pull/107 **Author:** [@freemjstudio](https://github.com/freemjstudio) **Created:** 8/22/2024 **Status:** ❌ Closed **Base:** `main` ← **Head:** `collection_name_param` --- ### 📝 Commits (1) - [`cfebc2a`](https://github.com/langchain-ai/langchain-postgres/commit/cfebc2aedc352bf1aa1a001455b9f9e2ba91d106) fix bug: collection name initialize in PGVector class ### 📊 Changes **1 file changed** (+5 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `langchain_postgres/vectorstores.py` (+5 -2) </details> ### 📄 Description ```python vectorstore = PGVector( embeddings=embeddings, collection_name=collection_name, connection=connection, use_jsonb=True ) ``` I found a small bug when I created PGVector class. I wanted to use Table that I already created but in original code, PGVector initializes collection_name with default collection name variable :_LANGCHAIN_DEFAULT_COLLECTION_NAME. I hope you check this out ! Thank you . --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 05:16:50 -05:00
yindo closed this issue 2026-02-16 05:16:50 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/langchain-postgres#169