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

Instrument tracing with langfuse #167

Conversation

strickvl
Copy link
Contributor

@strickvl strickvl commented Feb 18, 2025

  • add langsmith support for tracing on the app
  • formatting of the base code

@strickvl strickvl added enhancement New feature or request internal labels Feb 18, 2025
@strickvl strickvl requested a review from wjayesh February 18, 2025 12:05
Copy link

dagshub bot commented Feb 18, 2025

- Upgrade ZenML to version 0.74.0
- Add Pinecone, nest_asyncio, and asyncio to requirements
- Implement async version of get_completion_from_messages
- Add Langsmith callback for LLM requests
- Improve error handling for async completion
- Reorganize import statements in populate_index.py and llm_utils.py
- Remove redundant Pinecone import in populate_index.py
- Improve code formatting and import order
- Minor code cleanup and optimization
- Import os module to handle environment variables
- Add APP_ENVIRONMENT variable with default "dev"
- Include tracing tags in predict function with environment context
@strickvl strickvl changed the title Update for eval loop Instrument tracing with langfuse Feb 18, 2025
@strickvl strickvl marked this pull request as ready for review February 18, 2025 14:54
…omplete' into feature/langsmith-evals-deployment-updates
- Implement `get_langfuse_trace_id()` to fetch recent traces
- Update `vote()` function to log user feedback scores to Langfuse
- Add error handling and retry logic for trace retrieval
- Import necessary Langfuse and logging dependencies
@strickvl strickvl merged commit 8061aa6 into feature/pinecone-langsmith-llm-complete Feb 19, 2025
1 of 3 checks passed
@strickvl strickvl deleted the feature/langsmith-evals-deployment-updates branch February 19, 2025 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request internal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant