Skip to content
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

Move OpenTelemetry integration into core ai.jsx #231

Merged
merged 9 commits into from
Aug 9, 2023

Conversation

petersalas
Copy link
Contributor

  • Move OpenTelemetry render tracer implementation into AI.JSX
  • Prune memoized subtrees for legibility
  • Add OpenTelemetry logging integration
  • Add environment variable (AIJSX_ENABLE_OPENTELEMETRY) that enables the integration

The application (and/or deployment platform) is still responsible for using the OpenTelemetry SDK to connect it to a collector. In the examples repo, instrument.ts demonstrates how to do this (this can be imported at the top of any example to trace it).

@vercel
Copy link

vercel bot commented Aug 8, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ai-jsx-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 8, 2023 10:41pm
ai-jsx-nextjs-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 8, 2023 10:41pm
ai-jsx-tutorial-nextjs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 8, 2023 10:41pm

@NickHeiner
Copy link
Contributor

Can you update https://docs.ai-jsx.com/guides/observability for this?

@petersalas petersalas merged commit 652dcd5 into main Aug 9, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants