-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
feat: 🎸 source documents #1598
feat: 🎸 source documents #1598
Conversation
are kept only once
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Risk Level 2 - /home/runner/work/quivr/quivr/backend/llm/qa_base.py The code changes in this pull request are relatively safe, with a risk score of 2. The changes involve deduplication of source documents, which is a good practice to avoid redundancy. However, there are a few areas that could be improved for better code quality and maintainability:
🔍📝🪄 Powered by Code Review GPT |
🤖 I have created a release *beep* *boop* --- ## 0.0.107 (2023-11-06) ## What's Changed * fix: allow to change model bro brain settings tab by @mamadoudicko in #1590 * fix: fix notification banner display when too much items in chat list by @mamadoudicko in #1593 * docs: add auth modes config by @mamadoudicko in #1595 * fix: allow users to delete brains by @mamadoudicko in #1596 * feat: 🎸 source documents by @StanGirard in #1598 **Full Changelog**: v0.0.106...v0.0.107 --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
🤖 I have created a release *beep* *boop* --- ## 0.0.107 (2023-11-06) ## What's Changed * fix: allow to change model bro brain settings tab by @mamadoudicko in QuivrHQ/quivr#1590 * fix: fix notification banner display when too much items in chat list by @mamadoudicko in QuivrHQ/quivr#1593 * docs: add auth modes config by @mamadoudicko in QuivrHQ/quivr#1595 * fix: allow users to delete brains by @mamadoudicko in QuivrHQ/quivr#1596 * feat: 🎸 source documents by @StanGirard in QuivrHQ/quivr#1598 **Full Changelog**: QuivrHQ/quivr@v0.0.106...v0.0.107 --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
🤖 I have created a release *beep* *boop* --- ## 0.0.107 (2023-11-06) ## What's Changed * fix: allow to change model bro brain settings tab by @mamadoudicko in QuivrHQ/quivr#1590 * fix: fix notification banner display when too much items in chat list by @mamadoudicko in QuivrHQ/quivr#1593 * docs: add auth modes config by @mamadoudicko in QuivrHQ/quivr#1595 * fix: allow users to delete brains by @mamadoudicko in QuivrHQ/quivr#1596 * feat: 🎸 source documents by @StanGirard in QuivrHQ/quivr#1598 **Full Changelog**: QuivrHQ/quivr@v0.0.106...v0.0.107 --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
are kept only once
Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context.
Checklist before requesting a review
Please delete options that are not relevant.
Screenshots (if appropriate):