Skip to content

Conversation

ashwinb
Copy link
Contributor

@ashwinb ashwinb commented Oct 20, 2025

There should not be "vector db" anywhere.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Oct 20, 2025
@ashwinb ashwinb changed the title chore(rename): vector_db moved to vector_store everywhere chore(rename)!: vector_db moved to vector_store everywhere Oct 20, 2025
from llama_stack_client import Agent, AgentEventLogger, RAGDocument, LlamaStackClient

vector_db_id = "my_demo_vector_db"
vector_store_id = "my_demo_vector_store"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this demo script needs updating -- we should not be using rag_tool cc @ehhuang ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I posted a similar one, can update later

Copy link
Collaborator

Choose a reason for hiding this comment

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

draft here: #3870

has some other changes in it too.

@franciscojavierarceo
Copy link
Collaborator

Probably will break UI but I can fix that later as we haven't deployed it to NPM or anything so it's not a big deal

shields Manage safety shield services.
toolgroups Manage available tool groups.
vector_dbs Manage vector databases.
vector_stores Manage vector databases.
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: Manage vector stores

@ashwinb ashwinb changed the title chore(rename)!: vector_db moved to vector_store everywhere chore(cleanup)!: kill vector_db references as far as possible Oct 20, 2025
@ashwinb ashwinb marked this pull request as draft October 20, 2025 23:51
@ashwinb ashwinb marked this pull request as ready for review October 21, 2025 02:57
@ashwinb
Copy link
Contributor Author

ashwinb commented Oct 21, 2025

Probably will break UI but I can fix that later as we haven't deployed it to NPM or anything so it's not a big deal

Yeah almost certainly will break the UI.

@ashwinb ashwinb merged commit 122de78 into llamastack:main Oct 21, 2025
25 of 26 checks passed
@ashwinb ashwinb deleted the vector_db_nomo branch October 21, 2025 03:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants