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

update docsite embeddings automatically #197

Draft
wants to merge 1 commit into
base: assistant-docsite-rag
Choose a base branch
from

Conversation

hanna-paasivirta
Copy link
Contributor

@hanna-paasivirta hanna-paasivirta commented Mar 25, 2025

Short Description

Add
a) github action to update apollo embeddings when docsite is updated
b) Add a step to enable using the latest collection by default

Fixes #173

Implementation Details

a) todo:

  • set github action and script trigger
  • adjust naming convention to have more precise time (YYYYMMDDhhmm) in case there are multiple docsite udpates in a day
  • add step to then delete oldest collection if there's more than 3 collections with the name format "docsite-YYYYMMDD" (ignore other collections e.g. "docsite-test")

b) Accessing the latest collection:
Adds a step in the Search Docsite service to get the names of collections within an index, and use the collection with the latest date, unless a specific collection is not requested. This is done in _get_most_recent_namespace in DocsiteSearch.
When called from the AI assistant, this would run every time we query the database (once at the beginning of a conversation).

AI Usage

Please disclose how you've used AI in this work (it's cool, we just want to know!):

  • Code generation (copilot but not intellisense)
  • Learning or fact checking
  • Strategy / design
  • Optimisation / refactoring
  • Translation / spellchecking / doc gen
  • Other
  • I have not used AI

You can read more details in our Responsible AI Policy

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.

1 participant