Skip to content

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.

2 participants