Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
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

Lf #146

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

Lf #146

wants to merge 2 commits into from

Conversation

L-hhh
Copy link

@L-hhh L-hhh commented Jun 4, 2024

Add Vector Stores operation and add Vector Stores Files operation

@orhanerday
Copy link
Owner

Hi @L-hhh

Thank you so much for your contribution, could you please add vector introducing to the ReadMe? This would be so helpful for testing and introducing purpose.

Regards

@L-hhh
Copy link
Author

L-hhh commented Jun 5, 2024

Hi @L-hhh

Thank you so much for your contribution, could you please add vector introducing to the ReadMe? This would be so helpful for testing and introducing purpose.

Regards

Of course, vectors can be used together with Assistants. When creating Assistants, you can specify a vector file in the file_search of tool_resources, allowing your assistant to reference your vector file for searches. This will make the assistant's responses more aligned with your expectations. Additionally, Vector Store Files attach your Vector store to a file, enabling you to manipulate multiple vectors within a single file. I noticed that the current version only includes operations for Assistants and not for Vector stores, so I added operations for Vector stores.

Mat4B
Mat4B approved these changes Jun 10, 2024
@Mat4B
Copy link

Mat4B commented Jun 10, 2024

@orhanerday Can we merge it? Thanks.

Copy link

@mobweb mobweb left a comment

Choose a reason for hiding this comment

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

tested this locally and it works just as expected

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.

4 participants