-
Notifications
You must be signed in to change notification settings - Fork 28
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
feat: Add Langfuse Integration and Fix Audio Processing #355
Conversation
complain about Langfuse SDK's use of imports and mjs files
After deploying to staging and testing it using videos, I've got some errors. Staging bot is not working now. Going back to draft state. Issue #1: output stream closedSimilar to fluent-ffmpeg/node-fluent-ffmpeg#1135 , somehow Whisper will close connection early and triggers error on fluent-ffmpeg. It seems that the logic still works though, may consider ignoring the issue. Issue #2: AI reply bugSeems that we should fix CreateAIReply. The LLM call is mocked so that the change to the interface is not captured in the unit test. |
LANGFUSE_HOST is used by Python SDK, not TS ones
refactor: Update CreateAIReply tests to use getOpenAI()
This PR adds Langfuse integration and includes several fixes for audio processing and OpenAI-related functionality.
Main Changes
Langfuse Integration
OpenAI & Testing Updates
Upgraded OpenAI SDK, which broke a few things and need repairs.
Build & Configuration
The changes primarily focus on improving audio processing reliability and integrating Langfuse while maintaining test coverage and fixing various related issues.
Example Langfuse traces