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

Add Delete Function to Vector Store Drivers #533

Merged
merged 18 commits into from
Jan 12, 2024
Merged

Conversation

bjorger
Copy link
Contributor

@bjorger bjorger commented Dec 28, 2023

Partially implements #530

Add Delete Function to All Classes That Inherit from the Base Vectorstore Class
Add Delete Function to Base Vectorstore Class
Implement Delete Function for MongoDB Vectorstore
Implement Delete Collection Function for MongoDB Vectorstore
Add Test for Delete Function for MongoDB Vectorstore

Add Delete Function to Base Vectorstore Class
Implement Delete Function for MongoDB Vectorstore
Add Test for Delete Function for MongoDB Vectorstore
Add Delete Function to Base Vectorstore Class
Implement Delete Function for MongoDB Vectorstore
Add Test for Delete Function for MongoDB Vectorstore
Add Delete Function to Base Vectorstore Class
Implement Delete Function for MongoDB Vectorstore
Add Test for Delete Function for MongoDB Vectorstore
Add Delete Function to Base Vectorstore Class
Implement Delete Function for MongoDB Vectorstore
Add Test for Delete Function for MongoDB Vectorstore
Add Delete Function to Base Vectorstore Class
Implement Delete Function for MongoDB Vectorstore
Add Test for Delete Function for MongoDB Vectorstore
Add Delete Function to Base Vectorstore Class
Implement Delete Function for MongoDB Vectorstore
Add Test for Delete Function for MongoDB Vectorstore
pyproject.toml Outdated Show resolved Hide resolved
griptape/drivers/vector/base_vector_store_driver.py Outdated Show resolved Hide resolved
Add Delete Function to All Classes That Inherit from the Base Vectorstore Class
Add Delete Function to Base Vectorstore Class
Implement Delete Function for MongoDB Vectorstore
Implement Delete Collection Function for MongoDB Vectorstore
Add Test for Delete Function for MongoDB Vectorstore
To ensure, that delete_collection works as expected, the test was adjusted to guarantee that the behaviour is tested correctly.
griptape/drivers/vector/base_vector_store_driver.py Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
bjorgerSDA and others added 2 commits January 3, 2024 01:39
Rename delete to delete_vector
Remove google from poetry dependencies
Remove pyright from poetry dev dependencies
@collindutter collindutter changed the title Issue-530: Add Delete Function to Vectorstore Drivers Add Delete Function to Vector Store Drivers Jan 3, 2024
@collindutter
Copy link
Member

@bjorger FYI don't worry about the type-check action.

griptape/drivers/vector/base_vector_store_driver.py Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
Add return type to abstract delete_vector method
Remove new line from pyproject.toml
collindutter
collindutter previously approved these changes Jan 3, 2024
vasinov
vasinov previously requested changes Jan 3, 2024
griptape/drivers/vector/mongodb_vector_store_driver.py Outdated Show resolved Hide resolved
Remove delete_collection test
Copy link
Contributor Author

@bjorger bjorger left a comment

Choose a reason for hiding this comment

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

Work in review from @vasinov to remove delete_collections method

@collindutter collindutter requested a review from a team January 11, 2024 19:17
@collindutter collindutter dismissed vasinov’s stale review January 12, 2024 22:52

delete_collection was removed.

@collindutter collindutter merged commit bdb4e0d into griptape-ai:dev Jan 12, 2024
5 of 6 checks passed
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.

None yet

4 participants