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

docs: Add Qdrant migration code snippets #298

Merged
merged 11 commits into from
Jan 19, 2025

Conversation

dishaprakash
Copy link
Contributor

@dishaprakash dishaprakash commented Dec 30, 2024

docs: Add Qdrant migration code snippets

Test Log:

PyTest log
Log for pytest command

cd samples/migrations
pytest


(venv311) (base) ➜  migrations git:(qdrant-migration) ✗ pytest 
/usr/local/google/home/vishwarajanand/github/langchain-google-alloydb-pg-python/samples/migrations/venv311/lib/python3.11/site-packages/pytest_asyncio/plugin.py:208: PytestDeprecationWarning: The configuration option "asyncio_default_fixture_loop_scope" is unset.
The event loop scope for asynchronous fixtures will default to the fixture caching scope. Future versions of pytest-asyncio will default the loop scope for asynchronous fixtures to function scope. Set the default fixture loop scope explicitly in order to avoid unexpected behavior in the future. Valid fixture loop scopes are: "function", "class", "module", "package", "session"

  warnings.warn(PytestDeprecationWarning(_DEFAULT_FIXTURE_LOOP_SCOPE_UNSET))
===================== test session starts =====================
platform linux -- Python 3.11.9, pytest-8.3.3, pluggy-1.5.0
rootdir: /usr/local/google/home/vishwarajanand/github/langchain-google-alloydb-pg-python
configfile: pyproject.toml
plugins: asyncio-0.24.0, anyio-4.7.0
asyncio: mode=Mode.STRICT, default_loop_scope=None
collected 1 item                                              

test_qdrant_migration.py 
.                                                          [100%]

=================================== 1 passed in 57.51s ====================================
(venv311) (base) ➜  migrations git:(qdrant-migration) ✗

@dishaprakash dishaprakash requested review from a team as code owners December 30, 2024 11:48
@product-auto-label product-auto-label bot added the api: alloydb Issues related to the googleapis/langchain-google-alloydb-pg-python API. label Dec 30, 2024
Copy link
Collaborator

@averikitsch averikitsch left a comment

Choose a reason for hiding this comment

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

See comments from Chroma PR

Copy link
Collaborator

@averikitsch averikitsch left a comment

Choose a reason for hiding this comment

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

Please see any comments from Chroma PR

@vishwarajanand vishwarajanand changed the base branch from main to code-snippets January 17, 2025 21:28
Copy link
Contributor

@vishwarajanand vishwarajanand left a comment

Choose a reason for hiding this comment

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

LGTM

@vishwarajanand vishwarajanand merged commit 822528b into code-snippets Jan 19, 2025
8 checks passed
@vishwarajanand vishwarajanand deleted the qdrant-migration branch January 19, 2025 09:40
averikitsch pushed a commit that referenced this pull request Jan 22, 2025
* chore: add samples to migrate pinecone to alloy db

* fix: add Google file header

* fix: address PR comments

* fix: address pr comments

Changes:
1. Made snippets as standalone files
2. Compressed snippet functions into a single file.

* chore: address some pr comments

* fix: lint

* fix: lint

* fix: lint add type hints to params of main method

* chore: remove custom id column requirement

* chore: address pr comments

* fix: working pinecone snippets

* chore: update header year

* chore: lint fixes

* chore: tests skip adding test data if index exists

* chore: address pr comments

* docs: Add ChromaDB migration code snippets (#297)

* docs: Add Milvus migration code snippets (#299)

* docs: Add Qdrant migration code snippets (#298)

* docs: Add Weaviate migration code snippets (#300)

* chore: pr comments on sorted imports

* chore: add index_type in milvus index creation

---------

Co-authored-by: dishaprakash <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: alloydb Issues related to the googleapis/langchain-google-alloydb-pg-python API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants