mirror of
https://github.com/run-llama/create-llama.git
synced 2026-07-15 05:08:15 -04:00
LLMAgent doesn't support Ollama (needs LITS update) #92
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @joshuacox on GitHub (Oct 28, 2024).
Originally assigned to: @thucpn on GitHub.
How do I prevent openAI from being invoked at all?
.env
<question 1>
<question 2>
<question 3>
@marcusschiesser commented on GitHub (Oct 28, 2024):
@joshuacox we will upgrade llamaindex to fix this soon
@joshuacox commented on GitHub (Oct 28, 2024):
I deleted all initializers aside from Ollama in settings.ts after taking this lines advice:
Which I did to no effect. It still complains about the missing API key. I tried setting a false key to get it to ignore the issue, but of course openAI rejects the API call. But I do not want it contacting openAI at all.
EDIT: of course I submitted that comment and then the refresh happened and I see the comment from you @marcusschiesser
@mcavdar commented on GitHub (Oct 31, 2024):
hi @joshuacox
Can you please initialize with
npx create-llama@latest --prorather thannpx create-llama@latest, and update .env? This works for me.@marcusschiesser commented on GitHub (Nov 1, 2024):
should be fixed in Release 0.3.10 - please have a try
@mcavdar commented on GitHub (Nov 1, 2024):
hi @marcusschiesser
Unfortunately, updating LlamaIndex to version 0.8.2 didn't resolve the issue. If it would be helpful, I can share the differences between the files when initialized with --pro versus without it.
without-pro/app/api/chat/engine:
with-pro//app/api/chat/engine:
const agent = new OpenAIAgentI guess we need to fix this part.@marcusschiesser commented on GitHub (Nov 3, 2024):
@mcavdar great catch, it should be actually generate to use the
LLMAgentinstead of theOpenAIAgent- I fixed this in https://github.com/run-llama/create-llama/pull/410@marcusschiesser commented on GitHub (Nov 3, 2024):
Unfortunately, the Ollama LLM class from LlamaIndexTS doesn't support tool calls yet, I added this in https://github.com/run-llama/LlamaIndexTS/issues/1430
Once, this is added to LITS, it will automatically work in create-llama (after update). I keep this ticket open for tracking
@joshuacox commented on GitHub (Nov 11, 2024):
sorry for the delay, but this is working great for me now, closing. Feel free to re-open if there are some other issues.
@marcusschiesser commented on GitHub (Nov 12, 2024):
just for others, you need to update LllamaIndexTS if it doesn't work