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

Add TracingService and integration with Langsmith #2247

Merged
merged 32 commits into from
Jun 22, 2024
Merged

Conversation

ogabrielluiz
Copy link
Contributor

This pull request adds a new service called TracingService and a corresponding factory class called TracingServiceFactory. The TracingService is responsible for handling tracing functionality, while the TracingServiceFactory is used to create instances of the TracingService. This addition enables the application to support tracing capabilities.

Copy link

vercel bot commented Jun 21, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langflow ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 22, 2024 2:56am

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jun 21, 2024
@dosubot dosubot bot added the enhancement New feature or request label Jun 21, 2024
@ogabrielluiz ogabrielluiz marked this pull request as draft June 21, 2024 19:24
@ogabrielluiz ogabrielluiz marked this pull request as ready for review June 21, 2024 23:08
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Jun 21, 2024
This commit adds a new service called TracingService and a corresponding factory class called TracingServiceFactory. The TracingService is responsible for handling tracing functionality, while the TracingServiceFactory is used to create instances of the TracingService. This addition enables the application to support tracing capabilities.

Note: The commit message has been generated based on the provided code changes and recent commits.
🔧 (chat.py): Remove redundant setting of run_id in retrieve_vertices_order function
🔧 (chat.py): Remove setting of run_id in build_vertex function and add end_all_traces method call when no next_runnable_vertices
🔧 (base.py): Add logic to set run_name based on flow name and flow id, and end all traces with metadata in Graph class
…sistency and clarity in starter projects JSON files.

✨ (VectorStore-RAG-Flows.json): Update class inheritance to LCEmbeddingsModel for OpenAIEmbeddingsComponent to align with changes in the codebase
📝 (VectorStore-RAG-Flows.json): Add trace_type attribute to PromptComponent class for better tracking and identification of components

📝 (VectorStore-RAG-Flows.json): update import path for LCEmbeddingsModel in OpenAIEmbeddingsComponent to match new file structure and improve code organization.
…ed correctly with the required arguments

💡 (chat.py): improve error handling by passing error message to end_all_traces method and ensure it is called with the correct arguments
📝 (log.py): add LoggableType to define the type of log messages in the schema
📝 (schema.py): import LoggableType from log.py to use in defining the type of log messages in the Log class
📝 (service.py): add asyncio functionality to log_worker method for processing logs asynchronously
📝 (service.py): add start, flush, and stop methods to manage the logging worker task asynchronously
📝 (service.py): change initialize_tracers method to be asynchronous and start the logging worker
📝 (service.py): add async functionality to end method to stop the logging worker
📝 (service.py): change add_log method to be asynchronous and use asyncio to start the logging worker if not running
📝 (service.py): change contextmanager to asynccontextmanager for trace_context method to handle asynchronous operations
@ogabrielluiz ogabrielluiz merged commit 977181f into dev Jun 22, 2024
13 of 19 checks passed
@ogabrielluiz ogabrielluiz deleted the tracing_service branch June 22, 2024 03:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant