Aan Niraj Patel
Aan Niraj Patel
Created a draft PR (https://github.com/tmc/langchaingo/pull/801) with some open questions. Thoughts @tmc?
> Thanks so much for adding this! I need to dig into it to have a stronger opinion -- have you formed one in the meantime? I think the ctx...
> I was looking around to see if anybody was working on extending callback handlers with the ability to correlate start and end events, and my guess is that's also...
> I was looking around to see if anybody was working on extending callback handlers with the ability to correlate start and end events, and my guess is that's also...
I have pushed this commit for a review of the working direction as I work to fix the build: adding the OpenLLMetry module seems to not be playing nicely with...
Closing due to otel dependency version clash when installing OpenLLMetry - will need some additional exploration to address, opened #944 to introduce OpenTelemetry handler in the meantime.
Have we considered https://github.com/traceloop/openllmetry?
Ready for review. @tmc, @recht
Hi @codefromthecrypt, Thank you for taking the time to review! I've added some docs that talk about the example at docs/docs/modules/callbacks/opentelemetry/index.mdx (https://github.com/tmc/langchaingo/blob/1b9a058eb393df90fe5f2d6a08312bd1833e61e4/docs/docs/modules/callbacks/opentelemetry/index.mdx) - please let me know if I should...
> There is a summary thing I notice which is that we are adding attributes who are not required with zero values. This also applies to setting Status to `unset`....