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

feat(telemetry): Optimize trace storage and span processing logic #372

Closed
wants to merge 22 commits into from

Conversation

Imsharad
Copy link
Collaborator

🔄 Architecture and Tracing System Updates

Key Changes

  • Optimized tracing system:

    • Removed redundant raw context/response storage from traces table
    • Simplified schema by removing unnecessary constraints
    • Improved span processing logic in dbSpanProcessor.ts
  • Enhanced text generation:

    • Streamlined response handling across different model providers
    • Removed instrumentation overhead
    • Improved error handling and logging

Technical Details

  • Modified PostgreSQL schema:
    • Removed DEFAULT clauses for raw_context and raw_response
    • Simplified tracing schema
    • Optimized index creation
  • Updated C-3PO character config to use OpenAI instead of Anthropic
  • Added .gitignore entries for build directories and documentation

Dependencies

  • Updated is-weakref from 1.1.0 to 1.1.1

Documentation

  • Added detailed system architecture documentation
  • Included comprehensive Mermaid diagram for request flow visualization

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Imsharad! Welcome to the elizaOS community. Thanks for submitting your first pull request; your efforts are helping us accelerate towards AGI. We'll review it shortly. You are now an elizaOS contributor!

@Imsharad Imsharad closed this Feb 13, 2025
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.

2 participants