Skip to content

feat(go/genkit): Remove indexers for localvec #2862

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

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

Conversation

rutujaD1999
Copy link
Collaborator

@rutujaD1999 rutujaD1999 commented May 5, 2025

Description here... Help the reviewer by:

  • Issue : [Go] Remove Indexer as a primitive #2736
  • Remove dependency on indexer functionality provided by genkit repo from the localvec plugin
  • The indexer dependency is removed and a new helper indexer function is created in all the samples and unit tests to demonstrate an end to end flow.
  • Modified the docStore and dbvalue types to be able to access externally for further using it in helper index functions
  • Added getter and setter functions for docStore structure.
  • The dependency on indexer functions from genkit will be removed from remaining plugins in separate PRs.

Checklist (if applicable):

@rutujaD1999 rutujaD1999 requested a review from apascal07 May 5, 2025 09:38
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are you using this somewhere? doc-snippets is meant to be imported in docs and live docs aren't being read from here right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants