chore: Minor cleanup of vectorstore #371
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
chore: Minor cleanup of vectorstore All the review comments from the [cloud sql pr](googleapis/langchain-google-cloud-sql-pg-python#276) are being propogated here
product-auto-label
bot
added
the
api: alloydb
dishaprakash
added a commit
that referenced
this pull request
Mar 19, 2025
…402) * chore: Add Langchain standard vectorstore tests. (#356) * chore: Add Langchain standard vectorstore tests. * add header to test file * change header to 2025 * suppress warnings * chore: support ids in Documents and update insert SQL query to upsert (#361) * chore: support ids in Documents and update insert SQL query to upsert * remove pulling id from metadata * Fix previous vectorstore tests * Add comment * feat: Add get_by_ids and aget_by_ids to vectorstore (#364) * feat: Add get_by_ids and aget_by_ids to vectorstore * remove embedding column from being fetched * fix test * chore: Minor cleanup of vectorstore (#371) * chore: Minor cleanup of vectorstore chore: Minor cleanup of vectorstore All the review comments from the [cloud sql pr](googleapis/langchain-google-cloud-sql-pg-python#276) are being propogated here * Use `list` instead of `List` * remove unused import * Add sync test in async vectorstore * feat: Enable DB-Agnostic metadata filtering in vectorstores (#382) * feat!: Enable DB-Agnostic metadata filtering in vectorstores * Add header to util file * Minor change * Remove older filter tests * Change the data path * Linter fix * Remove async from func definition * Add sync test to async class * Remove table after test * Add fixture marker * Test * debug * debug * debug * debug * debug * debug * debug * debug * debug * debug * Linter fix * debug * debug * debug * Make non breaking change by supporting string filter * add text filter tests * Unify and/or operator * minor fix * minor fix * debug cloud build * debug cloud build * debug cloud build * debug cloud build * debug cloud build * debug cloud build * debug cloud build * debug cloud build * Debug * Make sure add documents does not alter original documents. * Minor change * Minor change * Review changes * Update test_vectorstore_search.py * Add extra test for code coverage * Add extra negative tests
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
chore: Minor cleanup of vectorstore
All the review comments from the cloud sql pr are being propogated here