Skip to content

Limit span resource attributes#1120

Merged
areibman merged 2 commits intomainfrom
23ck3v-codex/reduce-unnecessary-span-data
Jun 30, 2025
Merged

Limit span resource attributes#1120
areibman merged 2 commits intomainfrom
23ck3v-codex/reduce-unnecessary-span-data

Conversation

@areibman
Copy link
Contributor

Summary

  • skip recording zero-value token metrics
  • only attach system resource metadata on default session trace
  • clarify docs around zero-token filtering
  • test token usage filtering and resource attribute handling

Testing

  • pre-commit run --files agentops/instrumentation/common/token_counting.py agentops/sdk/attributes.py agentops/sdk/core.py tests/unit/instrumentation/common/test_token_counting.py tests/unit/sdk/test_resource_attributes.py
  • pytest tests/unit/sdk/test_resource_attributes.py tests/unit/instrumentation/common/test_token_counting.py -q

https://chatgpt.com/codex/tasks/task_e_685bc127ad5083218f326767573ec42c

@codecov
Copy link

codecov bot commented Jun 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@areibman areibman merged commit 033d0cc into main Jun 30, 2025
10 checks passed
@areibman areibman deleted the 23ck3v-codex/reduce-unnecessary-span-data branch June 30, 2025 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant