-
Notifications
You must be signed in to change notification settings - Fork 1.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
Add the LangChain integration #15229
Conversation
The |
The |
The |
a4d63bc
to
7a7051d
Compare
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.
Hey 👋 I had a quick first look and left some comments. Let us know if you need help to deploy your tile and sync the assets into staging/prod 🙂
The |
0d9173f
to
25e8b86
Compare
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.
Just left one more comment!
Also the tile is available on staging
Hi @FlorentClarret, could we get this merged? Thank you very much! |
What does this PR do?
Adds the LangChain integration, along with corresponding dashboard.
Additionally, this PR adds the
apm-core-python
andllm-obs
teams as codeowners for the OpenAI/LangChain integrations.Motivation
The LangChain APM integration (ref) adds tracing support for the Python LangChain library, along with metrics and logging for prompt-completions.
Additional Notes
Review checklist (to be filled by reviewers)
changelog/
andintegration/
labels attachedqa/skip-qa
label.