-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[experimental] Add command line tool for llama_index #1068
Conversation
amazing! will take a look soon |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
high-level looks good! main comment is about documenting usage
i like it - i can envision this being an easy-to-use CLI tool so that you can ask questions over any directory on your file system |
Got this error:
|
Hi @hophanms, "experimental" is not included in the package yet (I think it's why it called "experimental"). For now, you need to For long term, when this part of code is stable enough, we can move to src. At that time user will be able to call it directly with something like Hope it helps. |
I was in the wrong folder when I ran that CLI. I haven't digged further but tried this And got this:
There is no further way to debug. |
Hi @hophanms , could you provider some more information about your run? There're many reason can cause this output. Maybe I can describe the work flow clearly (maybe update the doc too, later).
Because we still call OpenAI API for index and question answering, environments like "OPENAI_API_KEY" still need to be set. Your output looks ok. There're no error message just no result too. For me, I can repro same result without "add" any file. I don't know if it's your case. |
I found out it silently failed on generating the .json so that's why the query returned |
Based on discussion #1054, add command line with following features:
The idea is quite straightforward, there're two files put in current directory
There is still a lot of work to be done, such as supporting various embedding models (even a local LLM maybe?), and removing/updating the current index. However, I am considering merging this fundamental part first so that I can tackle the project step by step.
Open to suggestions/changes.