Commit 4444bcf
authored
fix(go-adk): openai embeddings generation (#2132)
Close #2131
The main fix is to remove `HTTPClient` passed to embedding client so
that it creates its own and attaches auth header properly (one line
fix).
However, since we're here already, I'm also moving OpenAI, Azure, and
Ollama providers to use their respective SDK like Bedrock and Gemini
instead of doing direct HTTP calls to the endpoints. This ensures better
long-term stability and maintainability. Also improves some test
coverage + refactoring
---------
Signed-off-by: Jet Chiang <pokyuen.jetchiang-ext@solo.io>1 parent 42a656a commit 4444bcf
4 files changed
Lines changed: 460 additions & 173 deletions
File tree
- go/adk/pkg
- embedding
- memory
0 commit comments