Skip to content
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.

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

feat: add pgvector retriever support #6

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

shamspias
Copy link

This update introduces full support for the pgvector retriever in the configuration module, with specific enhancements to address metadata deserialization issues in asynchronous mode. The following changes were made:

  • Added make_pgvector_retriever to connect to a pgvector index.
  • Updated make_retriever to support the pgvector option.
  • Included async mode support in the PGVector initialization.
  • Enhanced PGVector Class: Overridden to ensure correct metadata deserialization when using async_mode=True. This directly addresses the metadata handling issue as detailed in issue #124 in the langchain-postgres repository.
  • Enhanced type annotations and error handling for better stability.
  • README updated to document pgvector support and usage instructions, including notes on handling metadata in async mode.
  • example.env updated with new environment variable examples required for pgvector configuration.
  • configuration retriever_provider updated with the pgvector option.

These updates enhance the robustness and functionality of the pgvector integration, ensuring reliable metadata handling and increasing flexibility and ease of use.

Related Issues

#5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant