-
Notifications
You must be signed in to change notification settings - Fork 5.5k
Adding monocle to traces doc #9112
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
|
@microsoft-github-policy-service agree [company="Okahu, Inc"] |
|
@hocokahu Thanks for your contribution. Having someone from the product team review this addition to the docs. |
@microsoft-github-policy-service agree [company="Okahu, Inc"] |
@microsoft-github-policy-service agree company="Okahu, Inc" |
docs/intelligentapps/tracing.md
Outdated
| | | Azure AI Inference | Foundry Agent Service | Anthropic | Gemini | LangChain | OpenAI SDK <sub>3</sub> | OpenAI Agents SDK | | ||
| |---|---|---|---|---|---|---|---| | ||
| | **Python** | ✅ | ✅ | ✅ ([traceloop](https://github.com/traceloop/openllmetry))<sub>1,2</sub> | ✅ | ✅ ([LangSmith](https://github.com/langchain-ai/langsmith-sdk))<sub>1,2</sub> | ✅ ([opentelemetry-python-contrib](https://github.com/open-telemetry/opentelemetry-python-contrib))<sub>1</sub> | ✅ ([Logfire](https://github.com/pydantic/logfire))<sub>1,2</sub> | | ||
| | **Python** | ✅ ([monocle](https://github.com/monocle2ai/monocle)) | ✅ | ✅ ([traceloop](https://github.com/traceloop/openllmetry), [monocle](https://github.com/monocle2ai/monocle))<sub>1,2</sub> | ✅ ([monocle](https://github.com/monocle2ai/monocle)) | ✅ ([LangSmith](https://github.com/langchain-ai/langsmith-sdk), [monocle](https://github.com/monocle2ai/monocle))<sub>1,2</sub> | ✅ ([opentelemetry-python-contrib](https://github.com/open-telemetry/opentelemetry-python-contrib), [monocle](https://github.com/monocle2ai/monocle))<sub>1</sub> | ✅ ([Logfire](https://github.com/pydantic/logfire), [monocle](https://github.com/monocle2ai/monocle))<sub>1,2</sub> | |
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.
Azure AI Inference is Microsoft's SDK, and it have 1st party support for instrumentation. Thus, I think we'd better not recommend the 3rd party monocle instrumentation.
docs/intelligentapps/tracing.md
Outdated
|
|
||
|  | ||
|
|
||
| ## Example 2: set up tracing with the Azure AI Inference SDK using Monocle |
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.
I think only one example to show the experience is good enough and there is no need to include three examples. @MuyangAmigo What's your opinion?
|
@hocokahu Can you please apply the feedback from the review? Thanks! |
|
@ntrogh @kuojianlu Yes, I am working on an update. |
…ode-docs into add-tracing-monocle
|
@ntrogh @kuojianlu I removed the 3P part under "Azure AI Inference" and previous Example 2. I still have OpenAI Agents SDK example as I haven't seen one but let me know if you guys want to keep it or remove it. |
Monocle is an open source by LF AI & Data Foundation. It is built on top of OpenTelemetry.
It does have support for most framework and models. More info at https://docs.okahu.ai/monocle_overview/