Add OpenTelemetry tracing via Sentry #38
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This adds OpenTelemetry tracing via Sentry to the base model class, which applies it to all models. It currently makes Sentry a peer dependency, which means we either need to do a major version bump and force all users into Sentry, or find a way to conditionally import Sentry and handle the case where it's not installed.
| ...mergedContext, | ||
| }); | ||
|
|
||
| return Sentry.startSpan({ name: spanName, op: 'llm' }, async (span) => { |
There was a problem hiding this comment.
I bet we could pas this in as an optional function in the constructor (telemetryWrapper or something) so that we don't need the sentry dependency in the library. (Or pass it as an argument to run).
There was a problem hiding this comment.
yeah, i think that is the correct approach. however, we could use this asap, so i'm going to release it as a major version bump with a warning and we will make it optional in a future major release.
There was a problem hiding this comment.
@brianbegy @cfortuner i released it as v3 but as a prerelase with the next tag so we can use it, but it won't be used for anyone that does npm install @dexaai/dexter.
This adds OpenTelemetry tracing via Sentry to the base model class,
which applies it to all models.
It currently makes Sentry a peer dependency, which means we either need
to do a major version bump and force all users into Sentry, or find a
way to conditionally import Sentry and handle the case where it's not
installed.
Example trace from Sentry:
