feat: add chat with models #2933
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…de/settings.json Update the editor.defaultFormatter setting in .vscode/settings.json to "charliermarsh.ruff" to improve code formatting for Python files. This change will ensure consistent code style across the project.
The code changes in `chat_llm_service.py` and `chat_llm.py` add functionality to save the answer for a chat with a model. This is achieved by updating the `save_answer` method in the `ChatLLMService` class and the `last_chunk` variable in the `ChatLLM` class. The changes include logging the chat ID and model name, as well as handling the last chunk of the response.
The code changes in `chat_llm.py` modify the `ChatLLM` class to filter the chat history and include only the messages that are relevant to the current question. This change improves the accuracy of the chat responses.
AmineDiro
reviewed
Aug 6, 2024
AmineDiro
reviewed
Aug 6, 2024
|
|
||
|
|
||
| @pytest.mark.base | ||
| def test_chat_llm(fake_llm): |
AmineDiro
reviewed
Aug 6, 2024
.vscode/settings.json
Outdated
There was a problem hiding this comment.
Do we need to commit this file ?
Co-authored-by: AmineDiro <aminedirhoussi1@gmail.com>
dosubot
bot
added
size:XL
This was referenced Aug 6, 2024
StanGirard
added a commit
that referenced
this pull request
Aug 7, 2024
🤖 I have created a release *beep* *boop* --- ## 0.0.294 (2024-08-07) ## What's Changed * Delete Porter Application quivr-com by @porter-deployment-app in #2927 * Delete Porter Application quivr-com-backend by @porter-deployment-app in #2928 * feat: quivr core tox test + parsers by @AmineDiro in #2929 * feat(frontend): handle no brain selection by @Zewed in #2932 * fix: processor quivr version by @AmineDiro in #2934 * fix: quivr core fix tests by @AmineDiro in #2935 * chore(main): release core 0.0.13 by @StanGirard in #2930 * feat: Add GitHub sync functionality to sync router by @chloedia in #2871 * refactor: Remove syncGitHub function from useSync.ts by @StanGirard in #2942 * feat: add chat with models by @StanGirard in #2933 * ci: precommit in CI by @AmineDiro in #2946 * feat: Add get_model method to ModelRepository by @StanGirard in #2949 * feat: Add user email to StripePricingOrManageButton and UpgradeToPlusButton components by @StanGirard in #2951 **Full Changelog**: v0.0.293...v0.0.294 --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
StanGirard
added a commit
that referenced
this pull request
Sep 10, 2024
🤖 I have created a release *beep* *boop* --- ## [0.0.14](core-0.0.13...core-0.0.14) (2024-09-09) ### Features * Add brain_id and brain_name to ChatLLMMetadata model ([#2968](#2968)) ([1112001](1112001)) * add chat with models ([#2933](#2933)) ([fccd197](fccd197)) * Add get_model method to ModelRepository ([#2949](#2949)) ([13e9fc4](13e9fc4)) * **anthropic:** add llm ([#3146](#3146)) ([8e29218](8e29218)) * **azure:** quivr compatible with it ([#3005](#3005)) ([b5f31a8](b5f31a8)) * **frontend:** talk with models and handle code markdown ([#2980](#2980)) ([ef6037e](ef6037e)) * quivr core 0.1 ([#2970](#2970)) ([380cf82](380cf82)) * using langgraph in our RAG pipeline ([#3130](#3130)) ([8cfdf53](8cfdf53)) ### Bug Fixes * **chat:** order of chat history was reversed ([#3148](#3148)) ([7209500](7209500)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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):