Commit 46b51fe
fix(embeddings): Ollama embeddings via native client; ollama in base deps
The default zero-config stack crashed on a lean install: `memorizz` ->
"langchain_ollama is required for Ollama embeddings". Two missing-dependency
problems:
1. The Ollama embedding provider used `langchain_ollama` (only in the `[ollama]`
extra). Rewrite it to use the native `ollama` client's embeddings endpoint —
the same SDK the LLM side already uses. langchain_ollama is dropped entirely.
2. The `ollama` client wasn't a base dependency, so even without (1) a bare
`pip install memorizz` couldn't run the default Ollama stack. Move
`ollama>=0.6.0` into base (small; httpx already comes via openai). No torch.
Why this slipped through testing: my dev env (memorizz_local) had
langchain_ollama installed (from the [all] extra), and I tested via that
interpreter rather than the shipped CLI's lean install — so the missing dep
never surfaced. Now verified in a clean 3.11 venv from the wheel: ollama present,
langchain_ollama/torch absent, build_session_agent + a 768-dim embedding work.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>1 parent c7c855d commit 46b51fe
3 files changed
Lines changed: 33 additions & 17 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
25 | 31 | | |
26 | 32 | | |
27 | 33 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
46 | 46 | | |
47 | 47 | | |
48 | 48 | | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
49 | 54 | | |
50 | 55 | | |
51 | 56 | | |
| |||
62 | 67 | | |
63 | 68 | | |
64 | 69 | | |
65 | | - | |
66 | | - | |
67 | | - | |
68 | | - | |
| 70 | + | |
| 71 | + | |
69 | 72 | | |
70 | 73 | | |
71 | 74 | | |
| |||
113 | 116 | | |
114 | 117 | | |
115 | 118 | | |
116 | | - | |
117 | 119 | | |
118 | 120 | | |
119 | 121 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
54 | 54 | | |
55 | 55 | | |
56 | 56 | | |
57 | | - | |
| 57 | + | |
58 | 58 | | |
59 | | - | |
| 59 | + | |
60 | 60 | | |
61 | 61 | | |
62 | | - | |
63 | | - | |
| 62 | + | |
| 63 | + | |
64 | 64 | | |
65 | 65 | | |
66 | | - | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
67 | 69 | | |
68 | | - | |
69 | | - | |
70 | | - | |
71 | | - | |
72 | | - | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
73 | 81 | | |
74 | 82 | | |
75 | 83 | | |
76 | 84 | | |
77 | 85 | | |
78 | 86 | | |
79 | 87 | | |
80 | | - | |
| 88 | + | |
81 | 89 | | |
82 | 90 | | |
83 | 91 | | |
| |||
126 | 134 | | |
127 | 135 | | |
128 | 136 | | |
129 | | - | |
| 137 | + | |
130 | 138 | | |
131 | 139 | | |
132 | 140 | | |
| |||
0 commit comments