Skip to content

Conversation

abdul-kabir-jawed
Copy link
Contributor

This PR fixes the incomplete tracing setup documentation by replacing a stub with comprehensive documentation.

Problem

The current docs/ref/tracing/setup.md file contains only:

# `Setup`

::: agents.tracing.setup

This provides no useful information to developers trying to understand how to configure tracing.

Solution

Replaced the stub with comprehensive documentation including:

  • Function Documentation: Detailed explanations for all setup functions:

    • set_tracing_disabled()
    • set_tracing_export_api_key()
    • add_trace_processor()
    • set_trace_processors()
  • Practical Examples: Working code examples for each function

  • Common Patterns: Real-world setup scenarios

  • Environment Variables: Documentation of OPENAI_AGENTS_DISABLE_TRACING

  • Troubleshooting: Common setup issues and solutions

Changes Made

  • docs/ref/tracing/setup.md: Complete rewrite with comprehensive documentation

Impact

  • ✅ Developers can now understand how to configure tracing
  • ✅ Clear examples for different setup scenarios
  • ✅ Troubleshooting guidance for common issues
  • ✅ Proper reference documentation for setup functions

@seratch
Copy link
Member

seratch commented Sep 15, 2025

Thanks for sending this. However, the ref files are generated from docstring in source code; so improve the docstring instead.

@seratch seratch added the documentation Improvements or additions to documentation label Sep 15, 2025
- Replace stub documentation in docs/ref/tracing/setup.md with comprehensive content
- Add detailed function documentation for all setup functions
- Include practical examples and common setup patterns
- Add troubleshooting section for common setup issues
- Document environment variables and configuration options

Fixes incomplete reference documentation that was previously just a module reference.
- Add detailed class and method descriptions
- Include practical usage examples
- Add implementation notes and best practices
- Document attributes and parameters
- Fix according to review feedback
@seratch seratch merged commit 31ed091 into openai:main Sep 15, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants