run-llama / llama-agents

MIT License
1.08k stars 80 forks source link

Installation Instruction missing one pip library #92

Closed peteryxu closed 2 days ago

peteryxu commented 6 days ago

started from scratch, cloned this repo, created an venv, and followed instruction doc below: pip install llama-agents

if you don't already have llama-index installed, to follow these examples, you'll also need

pip install llama-index-agent-openai

When running agentic_local_single.py, still get error below: ImportError: llama-index-embeddings-openai package not found, please run pip install llama-index-embeddings-openai

The Installation section of Readme.md should include the following: pip install llama-index-embeddings-openai

logan-markewich commented 6 days ago

fair enough, llama-index is defaulting to openai embeddings

Related to #87 this should be more lazily initialized I think

peteryxu commented 2 days ago

closed as fixed in PR#106