-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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: add knowledge_brain #2988
feat: add knowledge_brain #2988
Conversation
# 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. - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my code - [ ] I have commented hard-to-understand areas - [ ] I have ideally added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged ## Screenshots (if appropriate): --------- Co-authored-by: aminediro <aminedirhoussi1@gmail.com>
…r into release/feat/quivr-core-0.1
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Thank for the PR ! Some changes but overall very solid work and huge amount of changes !
backend/api/quivr_api/modules/sync/repository/sync_repository.py
Outdated
Show resolved
Hide resolved
backend/api/quivr_api/modules/knowledge/controller/knowledge_routes.py
Outdated
Show resolved
Hide resolved
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.
Thx for the changes. Minor changes to celery_worker and Embedding dim :+1
…r into release/fix/knowledge_brain
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.
Thx for the work !
I have made changes to the knowledge table schema and models to improve functionality and data organization. This includes adding new columns, modifying existing columns, and updating relationships. These changes will enhance the overall performance and usability of the application.