branch: externals/llm
commit 5271ce080920c714e3e7d013c22c9b1ae7e7f4da
Author: Andrew Hyatt <ahy...@gmail.com>
Commit: Andrew Hyatt <ahy...@gmail.com>
Fix issue where ollama sync embeddings always used localhost
This fixes https://github.com/ahyatt/llm/issues/34.
---
NEWS.org | 2 ++
llm-ollama.el | 2 +-
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/NEWS.org b/NEWS.org
index 3593d304ee..398a4f9b02 100644
--- a/NEWS.org
+++ b/NEWS.org
@@ -1,3 +1,5 @@
+* Version 0.12.1
+- Fix issue in =llm-ollama= with not using provider host for sync embeddings.
* Version 0.12.0
- Add provider =llm-claude=, for Anthropic's Claude.
* Version 0.11.0
diff --git a/llm-ollama.el b/llm-ollama.el
index 0ebe0dd2b6..b8224c3384 100644
--- a/llm-ollama.el
+++ b/llm-ollama.el
@@ -97,7 +97,7 @@ PROVIDER is the llm-ollama provider."
(cl-defmethod llm-embedding ((provider llm-ollama) string)
(llm-ollama--embedding-extract-response
- (llm-request-sync (format "http://localhost:%d/api/embeddings"; (or
(llm-ollama-port provider) 11434))
+ (llm-request-sync (llm-ollama--url provider "embeddings")
:data (llm-ollama--embedding-request provider string))))
(defun llm-ollama--chat-request (provider prompt)
