Skip to content

adding langfuse for openai #847

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Vaibhav159
Copy link
Contributor

Changes in This PR

Integration with Langfuse (Optional)

  • Added support for Langfuse as an optional add-on for enhanced telemetry and observability.
  • Created a fallback mechanism to ensure the core functionality is not blocked if Langfuse is not installed or configured.

Dummy observe Decorator

  • Introduced a placeholder observe decorator for seamless use in environments without Langfuse installed.
  • Ensures functions wrapped with @observe still execute normally when Langfuse is unavailable.
image

@CodeNameR
Copy link

interested in this

@ffesseler
Copy link

I would be surprised to see this merged as it introduces a direct dependency to a LLM platform. It would be better to add some telemetry with OpenTelemetry's standard libraries and possibly also those from openinference.

@marcklingen
Copy link

Langfuse maintainer here. Agree with @ffesseler that th best long term option would be to add native OTel instrumentation to pipecat in order to integrate well with multiple observability/evaluation projects. Happy to support this effort in case there are any questions.

@ffesseler
Copy link

@aconchillo Any guidelines on how to approach this? I would be interested in proposing an initial implementation and I would be curious to know if there are already ideas around the subject. (like what OTel instrumentation lib to use : openllmetry, openlit, openinference, ...?)

@marcklingen
Copy link

Created an issue to track an opentelemetry-based integration: #1456

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.

4 participants