llmware-ai / llmware

Unified framework for building enterprise RAG pipelines with small, specialized models
https://llmware-ai.github.io/llmware/
Apache License 2.0
6.62k stars 1.52k forks source link

Library.delete_installed_embedding(model_name, vector_db) modifies associated sqlite library record inconsistently #1094

Open wissamharoun opened 1 week ago

wissamharoun commented 1 week ago

environment: llmware v0.3.8, Apple Silicon macos 14.7, arm64, LLMWARE active_db set to sqlite, and vector_db set to chromadb

issue: after successfully installing embedding to an existing library

attempting to install a new embedding on the associated library (even if same embedding dims) typically fails, whereas attempting to delete the same embedding that was just installed also fails. Manually repopulating the value in the db with the initial one... ie: [{"embedding_status":` "no", "embedding_model": "none", "embedding_db": "none", "embedded_blocks": 0, "embedding_dims": 0, "time_stamp": "NA"}]

...is an effective but tedious workaround.

What is the correct and intended behavior of Library.delete_installed_embedding(model_name, vector_db)

?

doberst commented 5 days ago

@wissamharoun - sorry you ran into this issue - really appreciate you sharing all of the environment details - very helpful - let me replicate the environment and re-run the exact steps. It does sound like you have found a bug in the code. Will revert in the next 24-48 hours.