mirror of
https://github.com/unslothai/unsloth.git
synced 2026-08-11 09:54:09 +00:00
* Add customizable RAG embedding model setting and reorganize settings tabs Chat with files, project sources, and knowledge bases previously always embedded with unsloth/bge-small-en-v1.5. This adds a Settings option to pick any Hugging Face embedding model (or local path), with HF search autocomplete, server-side verification that the repo is actually an embedding model, and a save anyway escape hatch for offline or local models. The setting persists in app_settings and applies at runtime to both the sentence-transformers and llama-server GGUF embedder backends without a restart. Also reorganizes the General settings tab: Documents & RAG sits above Uploads, Helper LLM moved above the danger zone, and Model auto-switch (OpenAI API) moved to the bottom of the API tab. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Support local model paths on the GGUF embedder and normalize default saves Found by simulation testing of the embedding model setting: Local paths saved as the embedding model now work on the llama-server GGUF backend (the default backend on macOS and CPU). A path to a .gguf file is used directly and a directory is scanned for a variant-matching non-mmproj .gguf, with a clear error when none exists. Previously a local path was sent to the HF hub API and failed with a repo lookup error. Saving the default model explicitly no longer stores an override, so is_custom stays false and the UI does not show a reset button for the default value. * Address review: stale-vector handling, GGUF derivation, save-time guards Review follow-ups, each verified by new tests: Re-uploading a document after an embedding model change now re-indexes instead of deduping by content hash. Documents record the embedder that produced their vectors (lazy embedding_model column, NULL legacy rows keep deduping) and a mismatch replaces the old document. A vector width change no longer bricks the dense index. ensure_vec drops and recreates chunks_vec when the dim changes (old vectors are in a foreign space and only block inserts) and search_dense returns empty on a width mismatch instead of surfacing a vec0 error, so lexical search keeps working until documents are re-uploaded. Saving a local sentence-transformers folder with no .gguf now returns 409 with a clear message when the install embeds via llama-server, instead of failing at first index. force still saves. A custom RAG_EMBEDDING_MODEL env without RAG_EMBED_GGUF_REPO now derives the -GGUF companion repo instead of silently keeping the bge GGUF on CPU and macOS installs. The resolved GGUF path is tagged with the repo captured at entry, so a setting change during a download cannot mark the old model as current. GGUF repo detection matches gguf as a whole name segment rather than a substring, hf_token is trimmed before verification, and the settings combobox drops a redundant state mirror of its controlled value. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Shrink embedding model font to 11px in the input and dropdown The combobox wrapper applies className to the outer input group, so the size utility must target the inner input element; the previous text-xs never reached it and the field rendered at the browser default. * Show curated unsloth embedding models when the search field is empty The empty-query listing was the global top-downloads page, which holds no unsloth mirrors for the unsloth-first float to reorder, so the dropdown opened on third-party models. Match the model picker: curated unsloth listing when empty, whole-Hub search once a query is typed. * Address review: settings resilience and index consistency Keep the last known embedding model on settings store errors, remove the re-entrant dim lock in the llama-server backend, accept local GGUF saves and verify GGUF availability for HF repos on that backend, match local path embedders exactly in model list filters, drop same-width stale vectors from dense search, pin the embedder per ingestion job, and only replace completed documents after the re-index succeeds. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Consolidate the GGUF repo derivation tests * Trim to a single core embedding-model test * Address review: GGUF repo saves and cache race Accept a GGUF-named HF repo on the llama-server backend by verifying GGUF availability instead of the sentence-transformers metadata gate, and guard the settings cache with a generation counter so a read overlapping a save cannot repopulate it with the pre-save value. --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
124 lines
4.4 KiB
Python
124 lines
4.4 KiB
Python
# SPDX-License-Identifier: AGPL-3.0-only
|
|
# Copyright 2026-present the Unsloth AI Inc. team. All rights reserved. See /studio/LICENSE.AGPL-3.0
|
|
|
|
"""Persisted RAG embedding-model override (Settings -> General).
|
|
|
|
The stored value takes precedence over the ``RAG_EMBEDDING_MODEL`` env default in
|
|
``core.rag.config``. Vectors from different models live in different spaces, so
|
|
documents already indexed under the old model must be re-uploaded after a change
|
|
(the UI warns about this).
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import threading
|
|
import time
|
|
from typing import Any
|
|
|
|
EMBEDDING_MODEL_SETTING_KEY = "rag_embedding_model"
|
|
MAX_EMBEDDING_MODEL_LENGTH = 512
|
|
|
|
# The effective model is consulted on the embedder hot path (once per embed /
|
|
# tokenize call during ingestion), so the stored value is cached briefly instead
|
|
# of hitting sqlite each time. Writes invalidate immediately in-process; other
|
|
# readers converge within the TTL.
|
|
_CACHE_TTL_S = 2.0
|
|
_cached: tuple[float, str | None] | None = None
|
|
# Bumped on every write/invalidate. A reader captures it before the DB read and
|
|
# only fills the cache if it is unchanged afterward, so a read that overlapped a
|
|
# save cannot repopulate the cache with the pre-save value for the whole TTL.
|
|
_generation = 0
|
|
_lock = threading.Lock()
|
|
|
|
|
|
def _invalidate_cache() -> None:
|
|
global _cached, _generation
|
|
with _lock:
|
|
_cached = None
|
|
_generation += 1
|
|
|
|
|
|
def default_embedding_model() -> str:
|
|
"""The env/default model from rag config (``RAG_EMBEDDING_MODEL`` or bge)."""
|
|
from core.rag import config
|
|
return config.EMBEDDING_MODEL
|
|
|
|
|
|
def _coerce_embedding_model(value: Any) -> str | None:
|
|
if not isinstance(value, str):
|
|
return None
|
|
cleaned = value.strip()
|
|
if not cleaned or len(cleaned) > MAX_EMBEDDING_MODEL_LENGTH:
|
|
return None
|
|
# Newlines/control chars are never valid in a repo id or path.
|
|
if any(ord(ch) < 32 for ch in cleaned):
|
|
return None
|
|
return cleaned
|
|
|
|
|
|
def validate_embedding_model(value: Any) -> str:
|
|
cleaned = _coerce_embedding_model(value)
|
|
if cleaned is None:
|
|
raise ValueError(
|
|
"Embedding model must be a Hugging Face repo id (e.g. "
|
|
"'unsloth/bge-small-en-v1.5') or a local model path, up to "
|
|
f"{MAX_EMBEDDING_MODEL_LENGTH} characters."
|
|
)
|
|
return cleaned
|
|
|
|
|
|
def get_stored_embedding_model() -> str | None:
|
|
"""The persisted override, or None when unset/invalid."""
|
|
global _cached
|
|
now = time.monotonic()
|
|
with _lock:
|
|
cached = _cached
|
|
if cached is not None and now - cached[0] < _CACHE_TTL_S:
|
|
return cached[1]
|
|
gen = _generation
|
|
try:
|
|
from storage.studio_db import get_app_setting
|
|
stored = get_app_setting(EMBEDDING_MODEL_SETTING_KEY, None)
|
|
except Exception:
|
|
# Transient store failure: keep the last known value instead of
|
|
# silently reverting the embed/search hot path to the default model,
|
|
# which would mix vector spaces mid-ingestion.
|
|
with _lock:
|
|
if _cached is not None:
|
|
_cached = (time.monotonic(), _cached[1])
|
|
return _cached[1]
|
|
return None
|
|
value = _coerce_embedding_model(stored)
|
|
with _lock:
|
|
# Only cache when no save landed while we were reading; otherwise this
|
|
# value may be pre-save, and caching it would mask the new one for the
|
|
# TTL. The next reader re-reads the committed value.
|
|
if _generation == gen:
|
|
_cached = (time.monotonic(), value)
|
|
return value
|
|
|
|
|
|
def get_rag_embedding_model() -> str:
|
|
"""Effective embedding model: persisted override, else env/default."""
|
|
return get_stored_embedding_model() or default_embedding_model()
|
|
|
|
|
|
def set_rag_embedding_model(value: Any) -> str:
|
|
parsed = validate_embedding_model(value)
|
|
from storage.studio_db import upsert_app_settings
|
|
|
|
# Saving the default is not an override; keeps is_custom (and the UI's
|
|
# reset affordance) honest.
|
|
stored = parsed if parsed != default_embedding_model() else None
|
|
upsert_app_settings({EMBEDDING_MODEL_SETTING_KEY: stored})
|
|
_invalidate_cache()
|
|
return parsed
|
|
|
|
|
|
def reset_rag_embedding_model() -> str:
|
|
"""Clear the override; returns the (env/default) model now in effect."""
|
|
from storage.studio_db import upsert_app_settings
|
|
|
|
upsert_app_settings({EMBEDDING_MODEL_SETTING_KEY: None})
|
|
_invalidate_cache()
|
|
return default_embedding_model()
|