Commit 7af960e
authored
fix(deps): move tqdm, pyyaml, numpy from heavy extras into core (#118)
tqdm, pyyaml and numpy are light, pure-Python utilities that the umbrella
CLI (`pixelrag chunk`, `pixelrag index`) and the torch-free stage code
import at module load: chunk.py, index/config.py, and embed.py (which
deliberately lazy-imports torch but imports numpy/tqdm at the top).
Gating them behind the heavy `embed`/`index` extras forced a torch install
for light operations and made those modules fail to import on a core-only
install — which is why the test suite (deliberately torch-free; no test
imports torch/faiss/transformers) hit ModuleNotFoundError at collection
time on `uv sync --extra dev`.
Move the three into core dependencies and drop the now-redundant copies
from the embed/serve/index/eval extras. "Core stays light (no torch)"
still holds — these are not torch/GPU deps — and the suite now imports
cleanly with no skips or CI changes.1 parent 2d0ff8a commit 7af960e
2 files changed
Lines changed: 14 additions & 20 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
24 | 31 | | |
25 | 32 | | |
26 | 33 | | |
| |||
29 | 36 | | |
30 | 37 | | |
31 | 38 | | |
32 | | - | |
33 | | - | |
34 | 39 | | |
35 | 40 | | |
36 | 41 | | |
37 | 42 | | |
38 | | - | |
39 | 43 | | |
40 | 44 | | |
41 | 45 | | |
42 | 46 | | |
43 | 47 | | |
44 | 48 | | |
45 | | - | |
| 49 | + | |
46 | 50 | | |
47 | 51 | | |
48 | 52 | | |
| |||
52 | 56 | | |
53 | 57 | | |
54 | 58 | | |
55 | | - | |
56 | 59 | | |
57 | 60 | | |
58 | 61 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
0 commit comments