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 context filter to VectorPayload #92

Closed

Conversation

ronal2do
Copy link
Contributor

@ronal2do ronal2do commented Nov 11, 2024

based on this issue #91

Context

As a user I want to be able to use filter operations when using my chat

const result = await ragChat.chat<{ unit: string }>("Where is the capital of Japan?", {
    namespace,
    contextFilter: "unit = 'Samurai'",
});

Reference

https://upstash.com/docs/vector/features/filtering#metadata-filtering

@ronal2do ronal2do changed the title Cdd context filter to VectorPayload Add context filter to VectorPayload Nov 11, 2024
@CahidArda
Copy link
Contributor

Thanks a lot for the contributions @ronal2do!

It looks like the CIs can't run because your fork can't access the secrets we use in the CIs.

In order to use the secrets, we would need to update our CI. We would like to do this later.

In order to continue, we will close the PRs (this and #93) and open new ones with your commits.

Again, thanks a lot for the contributions!

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.

2 participants