Skip to content
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

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
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

DanielTea
Copy link

@DanielTea DanielTea commented Apr 19, 2023

  • Added a patch that allows users to upload multiple files at once, which improves the user experience by saving time and
    effort.

  • Implemented the ability to analyze multiple files simultaneously, providing more flexibility and convenience to the user.

  • The system now returns a confidence level for the answer provided, giving users a sense of the reliability of the information.

  • Implemented a feature to hide answers with low confidence levels, which ensures the accuracy and credibility of the results.

@mmz-001
Copy link
Owner

mmz-001 commented Apr 19, 2023

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
Copy link
Author

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
Copy link
Owner

mmz-001 commented Apr 20, 2023

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?

- Added multiple file upload feature using Streamlit file uploader
- Added loading bar using tqdm library to give feedback to the user on indexing progress
- Added section to sidebar displaying contributors to the project, retrieved using the GitHub API
- Parallelized document embedding function using concurrent.futures module
- Added progress bar using tqdm module to provide visual feedback on embedding progress
- reverted changes in the prompting
@DanielTea
Copy link
Author

DanielTea commented Apr 20, 2023

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

  • Added multiple file upload feature using Streamlit file uploader
  • Added loading bar using tqdm library to give feedback to the user on indexing progress
  • Added section to sidebar displaying contributors to the project, retrieved using the GitHub API
  • Parallelized document embedding function using concurrent.futures module
  • Added progress bar using tqdm module to provide visual feedback on embedding progress
  • reverted changes in the prompting--> I just removed the prompt changes.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants