LlamaIndexTS

LlamaIndexTS copied to clipboard

a starting point for a potential implementation of metadata filtering on simple vector store

⚠️ No Changeset found

Latest commit: ed30bee08a0d46bdeece744347ce49f5c265fa46

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
llama-index-ts-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 31, 2024 7:54pm

I'm hoping someone can take this as a starting point and finish the implementation from here. Fair warning what is present here might be garbage relative to the standards you guys are looking for.

@swiecki, that already looks very good! Great job, thanks a lot. Can you add a unit test and a changeset (just call pnpm changeset) - then I would be happy to merge it

bump @swiecki

metadataDict is used, but its:

1. not defined
2. never filled with content

closing as metadatafilters are already implemented for SimpleVectorStore - for usage check https://github.com/run-llama/LlamaIndexTS/blob/main/examples/metadata-filter/simple.ts