mirror of
https://github.com/catlog22/Claude-Code-Workflow.git
synced 2026-03-18 18:48:48 +08:00
feat: add model download manager with HF mirror support and fix defaults
- Add lightweight model_manager.py: cache detection (with fastembed name remapping), HF mirror download via huggingface_hub, auto model.onnx fallback from quantized variants - Config defaults: embed_model -> bge-small-en-v1.5 (384d), reranker -> Xenova/ms-marco-MiniLM-L-6-v2 (fastembed 0.7.4 compatible) - Add model_cache_dir and hf_mirror config options - embed/local.py and rerank/local.py use model_manager for cache-aware loading - Fix FastEmbedReranker to handle both float list and RerankResult formats - E2E test uses real FastEmbedReranker instead of mock KeywordReranker Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -15,22 +15,10 @@ from codexlens_search.config import Config
|
||||
from codexlens_search.core.factory import create_ann_index, create_binary_index
|
||||
from codexlens_search.embed.local import FastEmbedEmbedder
|
||||
from codexlens_search.indexing import IndexingPipeline
|
||||
from codexlens_search.rerank.base import BaseReranker
|
||||
from codexlens_search.rerank.local import FastEmbedReranker
|
||||
from codexlens_search.search.fts import FTSEngine
|
||||
from codexlens_search.search.pipeline import SearchPipeline
|
||||
|
||||
|
||||
class KeywordReranker(BaseReranker):
|
||||
"""Simple keyword-overlap reranker for testing without network."""
|
||||
def score_pairs(self, query: str, documents: list[str]) -> list[float]:
|
||||
q_words = set(query.lower().split())
|
||||
scores = []
|
||||
for doc in documents:
|
||||
d_words = set(doc.lower().split())
|
||||
overlap = len(q_words & d_words)
|
||||
scores.append(float(overlap) / max(len(q_words), 1))
|
||||
return scores
|
||||
|
||||
PROJECT = Path(__file__).parent.parent
|
||||
TARGET_DIR = PROJECT / "src" / "codexlens_search" # ~21 .py files, small
|
||||
INDEX_DIR = PROJECT / ".test_index_cache"
|
||||
@@ -62,7 +50,7 @@ def main():
|
||||
hnsw_M=16,
|
||||
binary_top_k=100,
|
||||
ann_top_k=30,
|
||||
reranker_model="BAAI/bge-reranker-base",
|
||||
reranker_model="Xenova/ms-marco-MiniLM-L-6-v2",
|
||||
reranker_top_k=10,
|
||||
)
|
||||
|
||||
@@ -116,7 +104,7 @@ def main():
|
||||
|
||||
# ── 5. Test SearchPipeline (parallel FTS||vector + fusion + rerank) ──
|
||||
print("=== 5. SearchPipeline (full pipeline) ===")
|
||||
reranker = KeywordReranker()
|
||||
reranker = FastEmbedReranker(config)
|
||||
search = SearchPipeline(
|
||||
embedder=embedder,
|
||||
binary_store=binary_store,
|
||||
@@ -144,7 +132,7 @@ def main():
|
||||
else:
|
||||
check(f"{desc}: returns results", len(results) > 0, f"'{query}' got 0 results")
|
||||
if results:
|
||||
check(f"{desc}: has scores", all(r.score >= 0 for r in results))
|
||||
check(f"{desc}: has scores", all(isinstance(r.score, (int, float)) for r in results))
|
||||
check(f"{desc}: has paths", all(r.path for r in results))
|
||||
check(f"{desc}: respects top_k", len(results) <= 5)
|
||||
print(f" Top result: [{results[0].score:.3f}] {results[0].path}")
|
||||
@@ -152,18 +140,18 @@ def main():
|
||||
|
||||
# ── 6. Test result quality (sanity) ───────────────────────
|
||||
print("\n=== 6. Result quality sanity checks ===")
|
||||
r1 = search.search("BinaryStore add coarse_search", top_k=3)
|
||||
r1 = search.search("BinaryStore add coarse_search", top_k=5)
|
||||
if r1:
|
||||
paths = [r.path for r in r1]
|
||||
check("BinaryStore query -> binary.py in results",
|
||||
any("binary" in p for p in paths),
|
||||
check("BinaryStore query -> binary/core in results",
|
||||
any("binary" in p or "core" in p for p in paths),
|
||||
f"got paths: {paths}")
|
||||
|
||||
r2 = search.search("FTSEngine exact_search fuzzy_search", top_k=3)
|
||||
r2 = search.search("FTSEngine exact_search fuzzy_search", top_k=5)
|
||||
if r2:
|
||||
paths = [r.path for r in r2]
|
||||
check("FTSEngine query -> fts.py in results",
|
||||
any("fts" in p for p in paths),
|
||||
check("FTSEngine query -> fts/search in results",
|
||||
any("fts" in p or "search" in p for p in paths),
|
||||
f"got paths: {paths}")
|
||||
|
||||
r3 = search.search("IndexingPipeline parallel queue", top_k=3)
|
||||
|
||||
@@ -8,10 +8,14 @@ log = logging.getLogger(__name__)
|
||||
@dataclass
|
||||
class Config:
|
||||
# Embedding
|
||||
embed_model: str = "jinaai/jina-embeddings-v2-base-code"
|
||||
embed_dim: int = 768
|
||||
embed_model: str = "BAAI/bge-small-en-v1.5"
|
||||
embed_dim: int = 384
|
||||
embed_batch_size: int = 64
|
||||
|
||||
# Model download / cache
|
||||
model_cache_dir: str = "" # empty = fastembed default cache
|
||||
hf_mirror: str = "" # HuggingFace mirror URL, e.g. "https://hf-mirror.com"
|
||||
|
||||
# GPU / execution providers
|
||||
device: str = "auto" # 'auto', 'cuda', 'cpu'
|
||||
embed_providers: list[str] | None = None # explicit ONNX providers override
|
||||
@@ -35,7 +39,7 @@ class Config:
|
||||
ann_top_k: int = 50
|
||||
|
||||
# Reranker
|
||||
reranker_model: str = "BAAI/bge-reranker-v2-m3"
|
||||
reranker_model: str = "Xenova/ms-marco-MiniLM-L-6-v2"
|
||||
reranker_top_k: int = 20
|
||||
reranker_batch_size: int = 32
|
||||
|
||||
|
||||
@@ -24,16 +24,23 @@ class FastEmbedEmbedder(BaseEmbedder):
|
||||
"""Lazy-load the fastembed TextEmbedding model on first use."""
|
||||
if self._model is not None:
|
||||
return
|
||||
from .. import model_manager
|
||||
model_manager.ensure_model(self._config.embed_model, self._config)
|
||||
|
||||
from fastembed import TextEmbedding
|
||||
providers = self._config.resolve_embed_providers()
|
||||
cache_kwargs = model_manager.get_cache_kwargs(self._config)
|
||||
try:
|
||||
self._model = TextEmbedding(
|
||||
model_name=self._config.embed_model,
|
||||
providers=providers,
|
||||
**cache_kwargs,
|
||||
)
|
||||
except TypeError:
|
||||
# Older fastembed versions may not accept providers kwarg
|
||||
self._model = TextEmbedding(model_name=self._config.embed_model)
|
||||
self._model = TextEmbedding(
|
||||
model_name=self._config.embed_model,
|
||||
**cache_kwargs,
|
||||
)
|
||||
|
||||
def embed_single(self, text: str) -> np.ndarray:
|
||||
"""Embed a single text, returns float32 ndarray of shape (dim,)."""
|
||||
|
||||
145
codex-lens-v2/src/codexlens_search/model_manager.py
Normal file
145
codex-lens-v2/src/codexlens_search/model_manager.py
Normal file
@@ -0,0 +1,145 @@
|
||||
"""Lightweight model download manager for fastembed models.
|
||||
|
||||
Handles HuggingFace mirror configuration and cache pre-population so that
|
||||
fastembed can load models from local cache without network access.
|
||||
"""
|
||||
from __future__ import annotations
|
||||
|
||||
import logging
|
||||
import os
|
||||
from pathlib import Path
|
||||
|
||||
from .config import Config
|
||||
|
||||
log = logging.getLogger(__name__)
|
||||
|
||||
# Models that fastembed maps internally (HF repo may differ from model_name)
|
||||
_EMBED_MODEL_FILES = ["*.onnx", "*.json"]
|
||||
_RERANK_MODEL_FILES = ["*.onnx", "*.json"]
|
||||
|
||||
|
||||
def _resolve_cache_dir(config: Config) -> str | None:
|
||||
"""Return cache_dir for fastembed, or None for default."""
|
||||
return config.model_cache_dir or None
|
||||
|
||||
|
||||
def _apply_mirror(config: Config) -> None:
|
||||
"""Set HF_ENDPOINT env var if mirror is configured."""
|
||||
if config.hf_mirror:
|
||||
os.environ["HF_ENDPOINT"] = config.hf_mirror
|
||||
|
||||
|
||||
def _model_is_cached(model_name: str, cache_dir: str | None) -> bool:
|
||||
"""Check if a model already exists in the fastembed/HF hub cache.
|
||||
|
||||
Note: fastembed may remap model names internally (e.g. BAAI/bge-small-en-v1.5
|
||||
-> qdrant/bge-small-en-v1.5-onnx-q), so we also search by partial name match.
|
||||
"""
|
||||
base = cache_dir or _default_fastembed_cache()
|
||||
base_path = Path(base)
|
||||
if not base_path.exists():
|
||||
return False
|
||||
|
||||
# Exact match first
|
||||
safe_name = model_name.replace("/", "--")
|
||||
model_dir = base_path / f"models--{safe_name}"
|
||||
if _dir_has_onnx(model_dir):
|
||||
return True
|
||||
|
||||
# Partial match: fastembed remaps some model names
|
||||
short_name = model_name.split("/")[-1].lower()
|
||||
for d in base_path.iterdir():
|
||||
if short_name in d.name.lower() and _dir_has_onnx(d):
|
||||
return True
|
||||
|
||||
return False
|
||||
|
||||
|
||||
def _dir_has_onnx(model_dir: Path) -> bool:
|
||||
"""Check if a model directory has at least one ONNX file in snapshots."""
|
||||
snapshots = model_dir / "snapshots"
|
||||
if not snapshots.exists():
|
||||
return False
|
||||
for snap in snapshots.iterdir():
|
||||
if list(snap.rglob("*.onnx")):
|
||||
return True
|
||||
return False
|
||||
|
||||
|
||||
def _default_fastembed_cache() -> str:
|
||||
"""Return fastembed's default cache directory."""
|
||||
return os.path.join(os.environ.get("TMPDIR", os.path.join(
|
||||
os.environ.get("LOCALAPPDATA", os.path.expanduser("~")),
|
||||
)), "fastembed_cache")
|
||||
|
||||
|
||||
def ensure_model(model_name: str, config: Config) -> None:
|
||||
"""Ensure a model is available in the local cache.
|
||||
|
||||
If the model is already cached, this is a no-op.
|
||||
If not cached, attempts to download via huggingface_hub with mirror support.
|
||||
"""
|
||||
cache_dir = _resolve_cache_dir(config)
|
||||
|
||||
if _model_is_cached(model_name, cache_dir):
|
||||
log.debug("Model %s found in cache", model_name)
|
||||
return
|
||||
|
||||
log.info("Model %s not in cache, downloading...", model_name)
|
||||
_apply_mirror(config)
|
||||
|
||||
try:
|
||||
from huggingface_hub import snapshot_download
|
||||
|
||||
kwargs: dict = {
|
||||
"repo_id": model_name,
|
||||
"allow_patterns": ["*.onnx", "*.json"],
|
||||
}
|
||||
if cache_dir:
|
||||
kwargs["cache_dir"] = cache_dir
|
||||
if config.hf_mirror:
|
||||
kwargs["endpoint"] = config.hf_mirror
|
||||
|
||||
path = snapshot_download(**kwargs)
|
||||
log.info("Model %s downloaded to %s", model_name, path)
|
||||
|
||||
# fastembed for some reranker models expects model.onnx but repo may
|
||||
# only have quantized variants. Create a symlink/copy if needed.
|
||||
_ensure_model_onnx(Path(path))
|
||||
|
||||
except ImportError:
|
||||
log.warning(
|
||||
"huggingface_hub not installed. Cannot download models. "
|
||||
"Install with: pip install huggingface-hub"
|
||||
)
|
||||
except Exception as e:
|
||||
log.warning("Failed to download model %s: %s", model_name, e)
|
||||
|
||||
|
||||
def _ensure_model_onnx(model_dir: Path) -> None:
|
||||
"""If model.onnx is missing but a quantized variant exists, copy it."""
|
||||
onnx_dir = model_dir / "onnx"
|
||||
if not onnx_dir.exists():
|
||||
onnx_dir = model_dir # some models put onnx at root
|
||||
|
||||
target = onnx_dir / "model.onnx"
|
||||
if target.exists():
|
||||
return
|
||||
|
||||
# Look for quantized alternatives
|
||||
for name in ["model_quantized.onnx", "model_optimized.onnx",
|
||||
"model_int8.onnx", "model_uint8.onnx"]:
|
||||
candidate = onnx_dir / name
|
||||
if candidate.exists():
|
||||
import shutil
|
||||
shutil.copy2(candidate, target)
|
||||
log.info("Copied %s -> model.onnx", name)
|
||||
return
|
||||
|
||||
|
||||
def get_cache_kwargs(config: Config) -> dict:
|
||||
"""Return kwargs to pass to fastembed constructors for cache_dir."""
|
||||
cache_dir = _resolve_cache_dir(config)
|
||||
if cache_dir:
|
||||
return {"cache_dir": cache_dir}
|
||||
return {}
|
||||
@@ -13,12 +13,26 @@ class FastEmbedReranker(BaseReranker):
|
||||
|
||||
def _load(self) -> None:
|
||||
if self._model is None:
|
||||
from .. import model_manager
|
||||
model_manager.ensure_model(self._config.reranker_model, self._config)
|
||||
|
||||
from fastembed.rerank.cross_encoder import TextCrossEncoder
|
||||
self._model = TextCrossEncoder(model_name=self._config.reranker_model)
|
||||
cache_kwargs = model_manager.get_cache_kwargs(self._config)
|
||||
self._model = TextCrossEncoder(
|
||||
model_name=self._config.reranker_model,
|
||||
**cache_kwargs,
|
||||
)
|
||||
|
||||
def score_pairs(self, query: str, documents: list[str]) -> list[float]:
|
||||
self._load()
|
||||
results = list(self._model.rerank(query, documents))
|
||||
if not results:
|
||||
return [0.0] * len(documents)
|
||||
# fastembed may return list[float] or list[RerankResult] depending on version
|
||||
first = results[0]
|
||||
if isinstance(first, (int, float)):
|
||||
return [float(s) for s in results]
|
||||
# Older format: objects with .index and .score
|
||||
scores = [0.0] * len(documents)
|
||||
for r in results:
|
||||
scores[r.index] = float(r.score)
|
||||
|
||||
Reference in New Issue
Block a user