-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add Tool Interaction Views #18
Conversation
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 you for the improvements; it does a non-breaking fix for what we want to do. In the long run, we should add a more extensible structure for custom views in the chat.
I would also defer to @philippzagar for some additional input.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #18 +/- ##
==========================================
+ Coverage 80.15% 80.69% +0.54%
==========================================
Files 14 15 +1
Lines 675 756 +81
==========================================
+ Hits 541 610 +69
- Misses 134 146 +12
Continue to review full report in Codecov by Sentry.
|
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.
Thanks for the PR @LeonNissen! 🚀
As @PSchmiedmayer already mentioned: We should definitely create a GitHub issue that lays out the future extension of Role
s into a more generalized setup.
Also, please add a small UI test to keep up the coverage tests that are failing
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.
Thanks @LeonNissen for this PR and adding the tests, looks graet! 🚀
@LeonNissen Just noticed that the commits on this branch are not properly signed (seems like a mismatch of the mail addresses), therefore preventing a merge. |
Add Tool Interaction Views
⚙️ Release Notes
With this release we add two new roles
.assistantToolCall
and.assistantToolResponse
which will be visible in the chat.This will provides the user with additional context about the underlying techniques/functions used during the chat.
📝 Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines: