Skip to content

Conversation

@teocns
Copy link
Contributor

@teocns teocns commented Jan 10, 2025

Problem

Users attempting to use agentops alongside autogen-core==0.4.0 encounter dependency resolution errors due to conflicting OpenTelemetry version requirements. This occurs because:

  • agentops supports Python ≥3.9 and uses OpenTelemetry 1.22.0
  • autogen-core==0.4.0 requires Python ≥3.10 and needs OpenTelemetry ≥1.27.0

Solution

Added version-specific OpenTelemetry constraints in pyproject.toml to handle both scenarios:

  • For Python 3.9: Pin OpenTelemetry packages to 1.22.0 (maintaining original behavior)
  • For Python ≥3.10: Allow OpenTelemetry ≥1.27.0 (compatible with autogen-core)

This ensures:

  1. Python 3.9 users get our tested OpenTelemetry versions
  2. Python ≥3.10 users can use agentops alongside autogen-core without conflicts

Testing

Verified dependency resolution works in both scenarios:

  • Python 3.9: Successfully installs with OpenTelemetry 1.22.0
  • Python 3.10 + autogen-core: Successfully upgrades to OpenTelemetry 1.27.0

No changes to core functionality or API.

@teocns teocns force-pushed the fix/otel-dependencies-compat branch from 9b8f424 to 44e74fb Compare January 10, 2025 22:58
@codecov
Copy link

codecov bot commented Jan 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Flag Coverage Δ
unittests 37.41% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@teocns
Copy link
Contributor Author

teocns commented Jan 10, 2025

tested this with python 3.9 and 3.13

@teocns teocns merged commit bfd6fd3 into main Jan 10, 2025
7 checks passed
@teocns teocns deleted the fix/otel-dependencies-compat branch January 10, 2025 23:04
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