mmz-001 / knowledge_gpt

Accurate answers and instant citations for your documents.
https://knowledgegpt.streamlit.app/
MIT License
1.57k stars 716 forks source link

Added support for uploading multiple files, implemented analysis for multiple files, returned confidence levels for answers, and added feature to hide low confidence answers. #33

Open DanielTea opened 1 year ago

DanielTea commented 1 year ago
mmz-001 commented 1 year ago

Thanks for the PR. I'll go through this and check if everything is fine before we can merge. (Might take some time though)

DanielTea commented 1 year ago

Thanks for the PR. I'll go through this and check if everything is fine before we can merge. (Might take some time though)

Hey thanks! I‘m not sure if we rather should merge it to dev first, cause I‘m a little inconfident if the prompt re engineering correct.

mmz-001 commented 1 year ago

Hey thanks! I‘m not sure if we rather should merge it to dev first, cause I‘m a little inconfident if the prompt re engineering correct.

After we've got everything working in this PR we can merge it to main.

What I suggest is that we split this PR into two, one for handling multiple files and one for the confidence levels so we can work on them separately. We could first work on the confidence level feature and move on to handling multiple files (which is the harder one to get right).

Can you split your changes into two branches and create two PRs?

DanielTea commented 1 year ago

Add multiple file upload, contributors, and parallelize embedding 05475ed

These changes should significantly improve the speed and usability of the KnowledgeGPT project.