community-experimental[minor]: feat: implement DatadogLLMObsTracer #5867
Conversation
dosubot
bot
added
the
size:XL
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
dosubot
bot
added
the
auto:enhancement
![dosubot[bot]](https://avatars.githubusercontent.com/in/324583?s=60&v=4){kind=small}
| @@ -0,0 +1,374 @@ | |||
| import { BaseCallbackHandlerInput } from "@langchain/core/callbacks/base"; | |||
There was a problem hiding this comment.
Hey there! I've reviewed the code and noticed that the new DatadogLLMObsTracer class includes a method persistRun that makes a new external HTTP request using the fetch API. I've flagged this change for your review to ensure it aligns with the project's requirements and best practices. Let me know if you have any questions or need further clarification!
| @@ -0,0 +1,374 @@ | |||
| import { BaseCallbackHandlerInput } from "@langchain/core/callbacks/base"; | |||
There was a problem hiding this comment.
Hey there! I've reviewed the code changes, and it looks like the diff adds and accesses environment variables using the getEnvironmentVariable function. I've flagged this for your review to ensure it aligns with the intended functionality. Let me know if you need further assistance!
| @@ -0,0 +1,383 @@ | |||
| import { test, jest, expect } from "@jest/globals"; | |||
There was a problem hiding this comment.
Hey team, just a heads up that this PR introduces new HTTP requests using the fetch function. I've flagged this for your review to ensure it aligns with our project's requirements and best practices. Let me know if you need any further assistance with this.
josemussa
force-pushed
the
feat/datadog-llmobs
branch
from
922f55f
to
5300306
Compare
jacoblee93
changed the title
|
Moved to |
|
Thanks again for this! |
This is an implementation of Datadog's LLM Observability API as a LangChain Tracer.
It provides support for the following Datadog spans: "llm", "tool", "workflow", "retrieval".