-
Notifications
You must be signed in to change notification settings - Fork 133
LangChain: Fix message timestamps, add default role assignment, and Bedrock support #1580
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
base: main
Are you sure you want to change the base?
Conversation
…t are recorded as LlmChatCompletionMessage event types.
…equest with Bedrock methods.
and having timestamp in there caused a problem. As a quick test, only add the request timestamp after the wrapped function has been invoked.
…nto sgoel/langchain-bug-fix
|
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1580 +/- ##
==========================================
- Coverage 81.82% 81.28% -0.54%
==========================================
Files 207 207
Lines 23953 23979 +26
Branches 3799 3804 +5
==========================================
- Hits 19599 19491 -108
- Misses 3086 3231 +145
+ Partials 1268 1257 -11 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Summary
This PR addresses several bugs related to LangChain integration in the New Relic Python Agent, specifically focusing on message timestamp handling and role assignments for chat completion events.
Changes
Message Timestamp Handling
LlmChatCompletionMessageeventsLlmChatCompletionSummarytable entriesRole Assignment
Bedrock Integration